Skip to content

fix(queens): restore composer puzzle navigation - #145

Merged
omercnet merged 1 commit into
mainfrom
queens/add-next-prev-pill-popover
Sep 21, 2026
Merged

omercnet merged 1 commit into
mainfrom
queens/add-next-prev-pill-popover

Conversation

@omercnet

Copy link
Copy Markdown
Owner

Summary

  • restore Previous and Next controls in the Queens composer popover
  • route popover navigation through the existing puzzle selection action with boundary-aware disabled states
  • refresh the composer documentation screenshot from the live 14×14 Hard popover

Verification

  • npm run check
  • npm run typecheck
  • npm test -- tests/composer-pill.test.ts
  • isolated UI: Next advanced puzzle 1 to 2; Previous returned to puzzle 1; first-puzzle Previous remained disabled
  • screenshot verified as a 2880×1800 PNG with one composer popover

Copilot AI lite review requested due to automatic review settings September 20, 2026 21:24

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@omercnet
omercnet merged commit 379e371 into main Sep 21, 2026
9 checks passed
@omercnet
omercnet deleted the queens/add-next-prev-pill-popover branch September 21, 2026 05:00
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.

2 participants