Repository navigation
Move player editing into a dialog, rework the mobile tabs, and add animations - #2
Merged
Merged
Conversation
…imations - Remove the game title from the UI (it stays in the data model). - Add a favicon. - Put Home, Edit players and fullscreen in one row in the header. - Edit players now happens in its own modal; the player list card is display-only. The modal keeps the keyboard up on touch screens and moves clear of it only when needed. - Remove the toast for undoing a new game. - Color picker: same layout as on phones, centered on its swatch, with a Done button on phones. - Mobile tabs: pressing the selected tab shows only the player list (its own "no tab selected" state, with the total scores), the table takes the list's room, and tabs slide, drop and resize with animation. - Player list: centers and smoothly scrolls to the active player, slides the round heading and scores on round changes, and bounces a score when saved. - Keypad keys show their press animation for keyboard input. - Table hint says "click" as it is only shown on desktop. Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Test plan
npm run lint,format:check,typecheck,test(327 tests) andbuildpassThe animations and keyboard behavior are not covered by the tests, as jsdom cannot show them.
🤖 Generated with Claude Code