Skip to content

Fix PingPong layout and camera recovery on landscape phones - #2

Merged
jorgesandev merged 1 commit into
mainfrom
fix/pingpong-landscape
Oct 10, 2026
Merged

jorgesandev merged 1 commit into
mainfrom
fix/pingpong-landscape

Conversation

@jorgesandev

Copy link
Copy Markdown
Owner

Horizontal phones previously stacked the camera below the score, retained the setup form's scroll position when starting a match, and paused on rotation without clearing calibration. The live view and score now sit side by side on short landscape screens, with pause/resume above both panels and secondary actions scrolling inside each panel. Long names fit the compact scoreboard, and dialogs respect viewport and safe-area limits.

Starting another screen resets page scroll. Both orientation APIs and changes to captured video dimensions now invalidate calibration, pause tracking and allow successful recalibration while retaining points. The video and calibration overlay share the same aspect ratio.

Validation: production build, formatting, lint, TypeScript and 18 unit tests pass. 41 browser tests pass across desktop, portrait and two landscape Chromium viewports (844×390 and 568×320), plus landscape WebKit. Tests cover synthetic camera calibration, rotation and frame-size changes, retained scores, pause/resume, detector controls, short dialogs, assisted point confirmation and the existing complete game flow. Physical camera/device and real-game accuracy checks remain pending.

@vercel

vercel Bot commented Oct 10, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
website Ready Ready Preview Oct 10, 2026 5:43pm UTC

@jorgesandev
jorgesandev merged commit d71b2f6 into main Oct 10, 2026
4 checks passed
@jorgesandev
jorgesandev deleted the fix/pingpong-landscape branch October 10, 2026 17:46

This branch was successfully deployed

1 active deployment
Preview — bfd90006 Deployed Oct 10, 2026 by vercel[bot]
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