Swap Books/Plan tab order; add segmented control to switch plans - #16
Merged
Conversation
Primary nav is now Plan, Books, History, Settings (was Books, Plan, History, Settings). Plan tab now shows an iOS-style segmented control above the day list for switching api.activePlanId() between api.plans() entries. Hidden when there's only one plan, since a toggle with a single, permanently -selected option isn't useful. It scrolls away with the page content rather than staying pinned, since ChapterGroup's own day headers are already sticky at top:0 in the same scroll container and that component is shared across the Books/Plan/History routes — pinning the toggle at the same offset would visually collide with them. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01K2wCTwwVhhkSDxQxWo7UoH
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
2 participants
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
PlanToggle, an iOS-style segmented control shown above the day list on the Plan tab, for switchingapi.activePlanId()betweenapi.plans()entries. Hidden when there's only one plan (a toggle with a single, permanently-selected option isn't useful).ChapterGroup's own day headers are alreadyposition: sticky; top: 0inside the same scroll container, and that component is shared across the Books/Plan/History routes, so pinning the toggle at the same offset would visually collide with the day headers once you scroll.Test plan
npm run buildpassestsc --noEmitpasses[/plan, /, /history, /settings]in that orderactivePlanId(confirmed the switch is visible in Settings' "Reading Plan — {name}" title too, not just local UI state), and is absent on the Books/History tabsGenerated by Claude Code