Repository navigation
style(web): theme-safe colours and one source for brand accents - #1991
Conversation
- Trace waterfall and span panel read DB and Cloudflare accents from a React-free @maple/ui module instead of two hand copies of the adapter maps. - One RAILWAY_ACCENT (integrations catalog) instead of two. - SQL query badges use text-primary-foreground on bg-primary, and the agent session tool input wash uses bg-muted/50 so it shows in dark mode.
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 6 minutes. View limit details
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 |
This comment has been minimized.
This comment has been minimized.
The same text-primary-foreground swap is in the opacity-ladder PR; keeping it in one place avoids a conflict on identical lines.
Maple review🟢 Confidence 10/10 · safe to merge. Pure constant/exports dedupe with identical values; the SQL-badge recolour that changed paint is no longer in the head. Moves the trace waterfall and span-panel brand accents into a React-free
|
🍁 Maple PR previewNote Preview resources were removed when the preview label came off. Final commit |


First pass from a dashboard consistency review. It fixes a colour that broke dark mode and removes duplicated brand-accent maps. Status colours, opacity levels and column widths are left to #1992, which covers them.
Changes
Tool input wash shows in dark mode. In an agent session's expanded tool call, the "Sent" half used
bg-black/[0.06], which disappears on a dark background. It now usesbg-muted/50, the most common wash in the app.One source for trace platform accents. The trace waterfall and the span panel each had a hand copy of the database brand colours and the Cloudflare orange, because the originals in
@maple/uiimport React icons. The colours now live in a React-free@maple/ui/lib/cloud-platforms/accents.ts, imported by all four consumers. The values moved unchanged, so this has no visual effect, and no lab fixture renders database spans in the waterfall to screenshot.One
RAILWAY_ACCENT, inintegrations/catalog.ts(it was also declared ininfra-integrations/kit/integration-empty.ts).The SQL query badge fix from the first version of this PR was dropped because #1992 makes the same change.
tool-io.tsalso has a one-line overlap with #1992; once #1992 merges, this wash becomesbg-surface-sunken(the same 50% muted).Verification
bun run typecheckpasses inapps/webandpackages/ui.cloud-platforms.test.ts(12 tests) passes, and so do the worst-case lab and shortcut tests that touch the trace views..oxlintrc.effect.json): no new findings on the changed files./lab/agent-session→ Transcript → expandread_file, captured headless at 2x againstorigin/mainand this branch, in light and dark.🤖 Generated with Claude Code