feat(front): rework the navigation, the account page, the Game page and My games #527
No reviewers
Labels
No labels
bug
documentation
duplicate
enhancement
good first issue
help wanted
invalid
question
spec
wayfinder:grilling
wayfinder:map
wayfinder:prototype
wayfinder:research
wayfinder:task
wontfix
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
amaurycoudr/chess-project!527
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feat/400-front-screens"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Summary
The four screens from the #399 prototype (navigation D, account page C, Game page C, My games D), plus the one contract change they need.
Shared pieces:
design-system/page-frame.tsx,design-system/ui/tabs.tsx,lib/routes.harness.tsx,tryChoiceOfanddateText.Evidence
rpc-game-readsandrpc-account-gamespage 41 Games that pair up on end time with none repeated or skipped. They filter by each result, keep the counts under a filter, and answerhasGamesfor a Guest, an Account and a browser with neither.site-navigation.test.tsx(new),app-shell,account-page(including a 375px French browser test of the phone tab row),game-pageandmy-games. Front: 1266 tests, green.my-games.specreaches My games from the navigation and filters it.accounts.specdeletes an Account through the Username menu and the dialog. The full suite ran locally, 39/39.room-closedtiming flake). After: see the pipeline on this head.Merge Danger
Door: two-way
Blast Radius: front-wide
game.listshape: it changed, and the front is its only consumer.router.test.tsx: edited despite the spec's "unchanged". Story 27 changes/account's signed-out heading; the/and/localassertions are untouched.account.mewhen its Game ends, so "My games" appears without a reload.site-frame.tsx: keeps its query and markup together, becausefront/view-is-compositionforbids a view passing<AccountNavView/>to its UI.Closes #400
Closes #401
Closes #402
Closes #403
Closes #404
Closes #405
added 3 commits
Compare with previous version
added 3 commits
Compare with previous version
added 1 commit
Compare with previous version
added 7 commits
mainbed61e4d- feat(front): head the account page with the Username, tab it, delete through a dialog2f4b2c5d- feat(api): page, filter and count a Player's Games, and say whether they own anye49a1b8a- feat(front): name the Game in the Game page's rail, and say Copied on its Copy linkd0489250- fix(front): lay the account page's tabs beside the section from mdCompare with previous version
added 1 commit
fd2d26da- feat(front): filter My games by result with counts, and page through every GameCompare with previous version
added 2 commits
00da4b69- feat(front): lead the navigation with New game, My games once owned, the account in a menua39d06f0- test(e2e): reach the account page from the Username menu before deleting the AccountCompare with previous version
Screenshots
Before is production, after is this merge request's review app at
484481e.Desktop
Mobile
added 6 commits
801eee8c- fix(front): keep an overflowing tab row from clipping its first tab9f6a463f- refactor(front): read the Account again at a Game's end from the Room view, given an options object584b8ec1- test(front): render the auth and games pages on one routes harness, and name the nav's placement93aae90d- refactor(front): share the account and My games page frame, and branch on unions exhaustively1e0802de- fix(front): keep the devtools trigger off the Room's Leave buttonc0a41f20- test(e2e): read the chosen filter by its role, and name Delete account onceCompare with previous version
changed the description
marked this merge request as ready
added 15 commits
main0f990f14- fix(front): lay the account page's tabs beside the section from md93cedf86- feat(front): filter My games by result with counts, and page through every Game72e84b12- feat(front): lead the navigation with New game, My games once owned, the account in a menuf298e4ff- test(e2e): reach the account page from the Username menu before deleting the Account58d7de47- fix(front): keep an overflowing tab row from clipping its first tabe60fd920- refactor(front): read the Account again at a Game's end from the Room view, given an options object86bf5c99- test(front): render the auth and games pages on one routes harness, and name the nav's placement7abece09- refactor(front): share the account and My games page frame, and branch on unions exhaustivelya5d7c300- fix(front): keep the devtools trigger off the Room's Leave button484481ee- test(e2e): read the chosen filter by its role, and name Delete account onceCompare with previous version
mentioned in merge request !537