Skip to content

[codex] Fix short-window layout overflow#4482

Open
jiaxindeyang-a11y wants to merge 1 commit into
esengine:main-v2from
jiaxindeyang-a11y:codex/fix-short-window-statusbar
Open

[codex] Fix short-window layout overflow#4482
jiaxindeyang-a11y wants to merge 1 commit into
esengine:main-v2from
jiaxindeyang-a11y:codex/fix-short-window-statusbar

Conversation

@jiaxindeyang-a11y

Copy link
Copy Markdown
Contributor

Summary

  • Let the sidebar and chat pane scroll vertically when the desktop window is shorter than the fixed chrome/footer stack.
  • Change the status bar from clipping overflow to horizontal scrolling so right-side metrics remain reachable in windowed layouts.
  • Update the typography overflow contract test to lock in the new behavior.

Root Cause

The desktop shell used fixed-height chrome/footer regions inside a body/layout that hides overflow. At very short heights, content outside the available column height was clipped. The status bar also forced a single line while using overflow: hidden, so metrics at the right edge disappeared when the window was not wide enough.

Validation

  • npm.cmd run check:css
  • npx.cmd tsx src/__tests__/typography-overflow-contract.test.ts
  • npm.cmd test
  • npm.cmd run test:typecheck was attempted but still fails locally because desktop/frontend/wailsjs is not generated in this checkout (Cannot find module '../../wailsjs/go/main/App').
  • Vite served http://127.0.0.1:5174/ with HTTP 200, but screenshot-level Browser verification was blocked by the local Windows sandbox (CreateProcessAsUserW failed: 5).

@github-actions github-actions Bot added desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development labels Jun 15, 2026
@jiaxindeyang-a11y jiaxindeyang-a11y marked this pull request as ready for review June 15, 2026 07:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

desktop Wails desktop app (desktop/**) v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants