Skip to content

Conversation

rachelks
Copy link
Member

@rachelks rachelks commented Sep 7, 2026

Summary

Closes #14.

  • Adds a props-driven BookJumpList component
  • Uses a native <select> wrapped in an accessible <label>
  • Uses currentIndex to reflect the currently selected book
  • Calls onSelect with the selected zero-based index
  • Adds responsive styling consistent with the existing app
  • Adds component tests using inline fixture titles only
  • Adds snapshot coverage
  • Does not fetch data or access the network

Validation

  • npm run format:check
  • npm run lint
  • npm test
  • npm run build

@rachelks rachelks added type: feature New functionality or requirement area: ui React components, presentation, navigation priority: high Required for first delivery status: in-review PR is open and awaiting review ai-assisted Substantial AI-generated code (documented per spec) labels Sep 7, 2026
@rachelks rachelks added this to the M2 — Core app milestone Sep 7, 2026
@rachelks rachelks requested a review from a team as a code owner September 7, 2026 14:03
@robertky robertky self-assigned this Sep 8, 2026
Copy link
Member

@robertky robertky left a comment

Choose a reason for hiding this comment

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

Looks good to me. The component is accessible, well tested, and matches the requirements in #14.

@robertky robertky merged commit b52e88c into main Sep 8, 2026
1 check passed
Sign in to join this conversation on GitHub.
Labels
ai-assisted Substantial AI-generated code (documented per spec) area: ui React components, presentation, navigation priority: high Required for first delivery status: in-review PR is open and awaiting review type: feature New functionality or requirement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

feat: BookJumpList (jump to a specific book)
2 participants