Skip to content

feat(1d4_web): improve game detail panel UX for motif navigation#1101

Merged
aaylward merged 1 commit intomainfrom
improve-game-detail-panel-ux
Mar 2, 2026
Merged

feat(1d4_web): improve game detail panel UX for motif navigation#1101
aaylward merged 1 commit intomainfrom
improve-game-detail-panel-ux

Conversation

@aaylward
Copy link
Copy Markdown
Collaborator

@aaylward aaylward commented Mar 2, 2026

Summary

  • Constrain motif list height (min(440px, 65vh) + overflow-y: auto) so it no longer extends far below the board
  • Add prev/next motif buttons under the board controls so you can step through interesting positions without scrolling the list at all
  • Show a motif counter (2 / 5) that updates as you navigate; shows total count when nothing is selected
  • Sort motif list by move order (ply) so display matches navigation sequence
  • Auto-scroll the active motif into view in the list when using the nav buttons
  • Works on mobile: 65vh cap prevents the list from eating the full screen when the layout wraps

Test plan

  • All 52 existing tests pass
  • New tests: prev/next buttons present when occurrences exist, absent when none
  • New tests: next motif seeks to first occurrence; prev wraps from initial state
  • New tests: motif counter shows total then index/total after navigation
  • New tests: motif list renders in ply order regardless of object key order
  • npm run typecheck clean
  • npm run build clean

- Add scrollable motif list with constrained height (min(440px, 65vh))
  so it no longer extends far below the board
- Add prev/next motif buttons under the board controls for stepping
  through interesting positions without scrolling the list
- Show motif counter (e.g. "2 / 5") that updates as you navigate
- Sort motif list by move order (ply) so display matches navigation
- Auto-scroll active motif into view when navigating with buttons
- Add tests for new motif navigation behavior
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Preview URL Updated (UTC)
✅ Deployment successful!
View logs
1d4-web c14475e Commit Preview URL

Branch Preview URL
Mar 02 2026, 09:27 PM

@aaylward aaylward merged commit ea27ef4 into main Mar 2, 2026
12 checks passed
@aaylward aaylward deleted the improve-game-detail-panel-ux branch March 2, 2026 21:28
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