Repository navigation
Mountain Weather on mobile: no sideways scroll, pinned tabs; matched temps and wind - #1495
Merged
Merged
Conversation
Contributor
|
Preview deployment: https://featxnwac-weather-temp-wind-tables.preview.avy-fx.org |
…cally The snow keys wrap instead of scrolling sideways. The section tabs and the weather summary's zones hang 1px below their row, which made both scroll vertically; they still swipe sideways. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Temps and wind share rows at xl, so they start and end level whatever their descriptions' lengths, with stripes and rules by date. On a phone every table shows one date at a time behind a toggle instead of scrolling sideways, and the issuance card drops its padding for the room. Wind cells read as the design has them: direction at full size, every speed bold, Calm bold after its direction. Lows are no longer bold. fmtWind had no callers left. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The temperature table always has room for its dates, so it skips the date switch the wider tables need. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rows The wording under each wind value set the row height of both paired tables. It moves to a shaded cell with a key, and stays as text for screen readers, so rows drop from 68px to 48px. The zone tables keep their wording. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
With the wording gone the two tables no longer need taller rows, so they drop to the 37px every other table uses, and lose their alternating colors. The wind key sits on one line that wraps on a phone, like the snow level key. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ggle under the table A forecast with no snow amounts shows one stretched 24-hour table instead of a small one behind a toggle. Elsewhere the 12h/24h toggle moves under the table, across from the snow key, and shrinks to fit that row. The wind key leads with the arrow and VAR notes, centered with the bar. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
rchlfryn
force-pushed
the
feat/nwac-weather-temp-wind-tables
branch
from
November 7, 2026 00:50
188b5d7 to
1d63db7
Compare
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…rows filled The section tabs pin below the mobile header, edge to edge, and keep the current tab scrolled into view; the current section is measured below them. Zone heading rows over the stations get a solid fill: bg-muted/50 rendered nothing, since the color variable holds a whole hsl() value. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
bg-muted/50 rendered nothing, since the color variable holds a whole hsl() value, so the header of the zone page's 6-hour blocks gets a solid fill. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The replace test renders a whole issuance twice and ran past Jest's 5s default when the pre-commit suite shared the machine; alone it takes under a second. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The gray fill blended into the table header, so the zone rows over the stations take the theme's primary color with its foreground text. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The test before this was not the only one: each renders the whole issuance, desktop and phone tables both, and another ran past 5s under the full suite. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…ywhere The dark band was too heavy, so zone rows become quiet uppercase labels. Desktop uses them too, in place of the separate Zone column, which goes with its code. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
… plain headings Reverts the small-caps rows, the dark band and the solid fill on the phone's zone rows, with the desktop Zone column and its code restored. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
|
🚀 This is included in version v1.27.0 |
This branch was successfully deployed
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.
Description
Makes the Mountain Weather page work on a phone and tightens up its tables. On a phone, nothing scrolls sideways any more: tables use a date toggle, the section tabs stay pinned under the header, and the page drops its card padding to give the tables room. On desktop, temperatures and wind sit level, wind strength is shown by shading, and the Extended section uses the same columns as temperatures and wind.
Related Issues
Feedback
Key Changes
Mobile fixes
bg-muted/50) rendered nothing, because the color variable holds a wholehsl()value. It now has a solid fill.Temperature and wind tables
Precipitation without snow amounts
How to test
/weather/forecast. Nothing should scroll sideways, every table except temperatures should have a date toggle.Screenshots / Demo video
Change dates to toggle for mobile view
Align bottom tables on desktop & add wind gradient

Migration Explanation
No migrations.
Future enhancements / Questions
WidgetZoneWeathertests get a 15s timeout. Each renders the whole issuance, desktop and phone tables both, and they ran past 5s locally when the whole suite ran in parallel. Each takes under a second on its own.bg-muted/50and still render without a fill. They're unchanged here.