Skip to content

Fix overlapping text in responsive window docs - #33

Merged
elithrar merged 1 commit into
mainfrom
fix/docs-responsive-window-header
Sep 5, 2026
Merged

elithrar merged 1 commit into
mainfrom
fix/docs-responsive-window-header

Conversation

@elithrar

@elithrar elithrar commented Sep 5, 2026

Copy link
Copy Markdown
Owner
  • Root cause: the Application window example sets a 760px maximum width but shrinks on mobile. Its header forced layout="inline", leaving the description in a column too narrow for its text. Text overlapped the Theme selector and Apply button at 320px and 390px.
  • Fix: let that header use the existing automatic container layout. The description stacks above the controls when narrow and stays beside them when space permits. Preserve the stacked window chrome and stacked action row.
  • Copy: label fixture widths “Up to …px” and remove the unconditional “above the breakpoint” caption, so the descriptions remain accurate on mobile.
  • Regression: extend the existing browser checks to detect rendered description text intersecting the actions, which the previous outer-frame containment checks missed. The new assertions fail on the original example at 320px and 390px.
  • Responsive validation: inspect before/after captures at 320, 390, 768, and 1280px; resize mounted examples through 16 additional widths from 360 to 1440px, including breakpoint neighbors. No text/control overlap or document overflow remains.
  • Checks: npm run check passes (54 unit tests), all 24 browser tests pass, and library/docs builds pass. Existing menu, select, autocomplete, and overlay geometry regressions pass. Browser validation used Chromium; Safari/WebKit was not separately exercised.

The change is limited to the existing docs example and its browser regressions.

@elithrar
elithrar merged commit 28065a5 into main Sep 5, 2026
1 check passed
@elithrar
elithrar deleted the fix/docs-responsive-window-header branch September 5, 2026 09:06
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