Repository navigation
Short screens: keep controls reachable and focus visible below dialog titles - #80
Conversation
… titles Players can reach controls on very short screens and keep keyboard focus visible below dialog titles. Written by Codex. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Replace the external Playwright dependency with the existing Chrome DevTools helpers while preserving native keyboard traversal, clipping, hit ownership and all viewport/font/notice cases. Skip only when Chrome is unavailable and report completed checks in TAP diagnostics. Written by Codex.
|
737ea42 ports the short-height browser test from Playwright to the repo's own CDP helper, so it now runs in CI. It skips only when Chrome is missing. All checks are kept. Results: locally 1 passed (12 cases, 666 checks). With the close-button layout deliberately broken it fails with 21 geometry failures. The standalone baseline mode passes 224 checks. 🤖 Generated with Claude Code |
…layout # Conflicts: # scripts/test-selection.mjs
…layout # Conflicts: # scripts/test-selection.mjs
What changes for players
Players can reach controls on very short screens and keep keyboard focus visible below dialog titles.
Checks
Written by Codex (Cloud job
p-w12-high-zoom-reflow, START a781c22). Claude reviewed it, ran the focused tests locally and wrote the notes above.🤖 Generated with Claude Code