Skip to content

Fix localized homepage title overflow on mobile - #113

Merged
LakshmanTurlapati merged 1 commit into
mainfrom
fix-mobile-title-overflow
Aug 10, 2026
Merged

Fix localized homepage title overflow on mobile#113
LakshmanTurlapati merged 1 commit into
mainfrom
fix-mobile-title-overflow

Conversation

@LakshmanTurlapati

@LakshmanTurlapati LakshmanTurlapati commented Aug 10, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • constrain the homepage hero content to the available viewport width
  • allow longer localized titles to wrap and balance instead of forcing one English-sized line
  • add regression checks that reject a future white-space: nowrap reintroduction

Root cause

The mobile hero forced every locale onto one line and sized the title using an English-specific width estimate. German, Spanish, and unbroken CJK text could exceed the content box, while the hero clipped the overflow.

FSB mobile validation

Validated the prerendered localized homepages through FSB in exact 390 × 844 CSS-pixel viewports:

Locale Title lines Viewport Document width Result
English 1 390px 390px Fits
Spanish 2 390px 390px Fits
German 2 390px 390px Fits
Japanese 2 390px 390px Fits
Simplified Chinese 1 390px 390px Fits
Traditional Chinese 1 390px 390px Fits

Every locale reported scrollWidth === clientWidth === 390 and title bounds fully inside the viewport.

English, Spanish, and German

FSB mobile title validation for English, Spanish, and German

Japanese, Simplified Chinese, and Traditional Chinese

FSB mobile title validation for Japanese, Simplified Chinese, and Traditional Chinese

Verification

  • SKIP_BUILD=1 node tests/showcase-build-smoke.test.js — 86 passed
  • node tests/showcase-angular-foundation.test.js — 51 passed
  • npm --prefix showcase/angular run build — 54 localized routes prerendered
  • npm --prefix showcase/angular run verify:hreflang — 541 passed

@LakshmanTurlapati
LakshmanTurlapati marked this pull request as ready for review August 10, 2026 16:29
@LakshmanTurlapati
LakshmanTurlapati merged commit 09d95a6 into main Aug 10, 2026
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant