fix(theme): light-mode contrast stragglers (DEVNET pill, dim chips, dashboard preview) - #2432
Conversation
- DEVNET header pill: var(--warning) instead of hardcoded amber-400 (#fbbf24 is ~1.7:1 on the light theme's white header) - Trade utility row: address chip, copy button, Analytics link, usd/tokens toggle, and the mobile sheet close button move from --text-dim (ghost decoration tier, ~1.5:1) to --text-muted — they carry information or are interactive, so they shouldn't sit in the deliberately-invisible tier - Dashboard connect-gate preview: crisp ghost (opacity only) instead of blur-sm, which dissolved into shapeless smudges on the light theme Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
1 participant
Item 9 from the live-site UX sweep (item 10 — the LONG tab white-on-green — was refuted on inspection: it's already
text-black; the locked fieldset's 40% opacity just made it look white in screenshots).var(--warning)(was hardcoded#fbbf24, ~1.7:1 on white). Readable amber-700 in light, unchanged look in dark.--text-dim→--text-muted. These are interactive/informative, not ghost decoration; the six remaining--text-dimuses on the page are genuine fine-print/icons and stay.blur-sm, keepopacity-40— the blurred near-white panels dissolved into smudges on the light theme that read as a rendering glitch.tsc clean.
🤖 Generated with Claude Code