Put the week on the resort page, and fold the detail cards - #14
Merged
Merged
Conversation
The resort page answered "how is it now" and nothing else: finding the best day for one mountain meant switching to the planner and hunting for its row. And on a phone it was seventeen cards in one column, about five screens, all at the same volume. - New ResortWeekStrip under the summary: seven days with score, a snow bar and the best day outlined. It reads the planner's mid-mountain outlook and cache, so a day scores the same here as in the grid. - The detail cards are grouped into collapsible sections (Snow, Wind & visibility, Temperature & humidity, Forecast discussion). Each header keeps its headline number visible. On a phone Snow starts open, and Wind or Temperature open themselves on a hold or frostbite day. Desktop starts with everything open. - Desktop uses a two-column layout: the summary and the week stay in a sticky left column while the details scroll on the right. Phone page height for Vail drops from ~4,100px to ~2,950px with only the Snow section open. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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
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.
Why
What
ResortWeekStrip(new), shown under the Today summary: 7 days, each with its score, a snow bar scaled to the week's snowiest day and the snow amount. The best day is outlined. It usesusePlanner([resort]), which reads the planner's mid-mountain outlook and shared cache, so each day scores the same as it does in the Planner grid.DashboardSection(new): a collapsible group built on native<details>, with a one-line hint in the header so a closed section still shows its headline number. Sections:Desktop (≥1024px) starts with every section open. A closed section's cards aren't rendered at all.
Desktop two-column layout: the summary and week strip sit in a sticky left column while the details scroll on the right. The
TodaySummarytiles show 2 across in that column and 4 across on tablets.Pro View: unchanged.
Testing
tsc --noEmit,next lintand all 9 offline test suites pass.Merge order
This is 2 of 3. It branches off
mainand only editscomponents/(WeatherDashboard,TodaySummary, plus the two new components), so it merges cleanly before or after #13.🤖 Generated with Claude Code