Skip to content

style(web): one scale for type, icons, radii, hover, borders and timeline colours - #1988

Merged
Makisuo merged 3 commits into
ui-consistency-3from
ui-consistency-4
Oct 30, 2026
Merged

Makisuo merged 3 commits into
ui-consistency-3from
ui-consistency-4

fix(web): timeline and namespace mixes in oklab, 2xx on the healthy t…

650e71a
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 30, 2026 in 1m 8s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. The five files changed since 47cc281 are class and colour-token swaps; every token they name is defined and resolved by the page's theme Mount.
no findings · tests not needed · risk low

Re-review of the five files changed since the last pass: 2xx moves to --severity-info on both DOM and canvas, timeline and namespace fills become color-mix washes, hovers go to muted/50. No defects. The open time-range-picker TypeError is not addressed; those lines only change border opacity.

  • STATUS_COLORS["2xx"] and the 2xx mix bar use --severity-info instead of dark-only --chart-3
  • barFillFromHue washes the bar hue into --background via color-mix(in oklab, …)
  • Namespace group labels mix their hue with --foreground/--background instead of fixed dark values
  • view-api.ts and endpoint-peek rows hover at muted/50 instead of /40

Production impact

Open errors in the changed files (1)
Issue Service Occurrences File
TypeError maple-web 3 apps/web/src/page/kit/time-range-picker.ts

After this merges, Maple checks whether they stop.

What was checked
  • --severity-info was added to API_THEME_TOKENS, and tokenProperty unwraps var(--severity-info) (charts/theme-colors.ts:19), so the stack and the peek resolve it
  • bar.fill/bar.borderColor land only in inline DOM styles (timeline/view-row.ts:290-291), where color-mix with var(--background) resolves
  • --chart-3 has no remaining reader in the services detail page or its charts, so dropping it from the token list strands nothing

650e71a · Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.