Restore the fuller layout dropped by the history rewrite, plus consistent button scale - #12
Merged
Merged
Conversation
Every page sat in a 700px column at laptop widths, leaving most of the screen empty. Content containers widen to 1100px (settings 900, session screens 900 since questions are a reading surface), cards get larger padding and type, and lists of peer items become responsive grids that collapse to one column on phones: today's sessions and achievement badges on the dashboard, sessions within a day on the plan, quick actions and learning path steps on Learn, badge cards on achievements, deck and guide cards, and settings sections as label and control rows. The plan form widens to 720 with paired fields, the activity heatmap stretches to the container, the runner's question text and controls scale up, preflight stats and end screen stats sit four across on desktop, and the chat thread fills a 900px column with a taller composer. Grids use auto fit so a pair of cards splits the full width instead of leaving an empty track. All README screenshots and the answer loop GIF are recaptured at the new width at double resolution for crisp rendering.
Matches every page's buttons to the session screens' scale: filled primaries around 0.7rem by 1.4rem padding at 1rem font, secondaries slightly smaller, chips kept compact but comfortable. Covers the dashboard Start and New Plan buttons, plan header actions, Learn's start button, settings Save and Detect, flashcards and guides actions, chat send, and auth submits.
The previous captures ran against a build that was missing the fuller layout after a history rewrite briefly reverted it, so they showed the narrow containers and small buttons. These are taken against the restored layout with the enlarged buttons, and the dashboard shot now has a pending session so the filled Start button is visible.
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
1 participant
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.
The history rewrite that scrubbed attribution was based on a stale local ref and accidentally dropped the fuller-layout merge from main. This restores it and finishes the button work on top:
Verification
tsc --noEmitclean, 646 unit tests passedGenerated by Claude Code