fix(alert): readable dark-gold alert card with capped scroll, tucked timestamp, breakable URLs - #118
Open
JamieRuderman wants to merge 2 commits into
Open
fix(alert): readable dark-gold alert card with capped scroll, tucked timestamp, breakable URLs#118JamieRuderman wants to merge 2 commits into
JamieRuderman wants to merge 2 commits into
Conversation
…text, move timestamp out of the way
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
smart-trip-web | 55e8567 | Commit Preview URL Branch Preview URL |
Jul 13 2026, 07:17 PM |
…, shrink scroll, break long URLs
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.
Problem
Long SMART service alerts (e.g. the Geyserville-station notice) took over the
whole screen, the gold text was low-contrast and used several inconsistent
shades, the start timestamp floated against the wrapping title, and the long URL
couldn't break — orphaning "at" on its own line.
Changes
ServiceAlert.tsx
max-h-32 overflow-y-auto overscroll-contain— long alerts scroll inside the card instead of pushing the page down.
new
text-smart-gold-dark(no more mixed/60,/70,/80opacities). Deepenough to read clearly on the pale card in light mode.
time sits tucked directly under the close button, right-aligned and out of the
text flow (previously it collided with the ✕ / floated mid-title).
break-wordson the title and message letssonomamarintrain.org/geyservillewrap, removing the orphaned "at".Theme (index.css, tailwind.config.ts)
--smart-gold-darktoken: deep gold34 74% 38%in light mode, brightenedto
40 85% 62%in dark mode so the same class stays legible on the dark card.Verification
readable, message scrolls internally at the smaller height, ✕ sits top-right
with the timestamp tucked beneath it, the URL wraps with no "at" orphan, and no
horizontal overflow. No console errors.
🤖 Generated with Claude Code