Skip to content

style(web): neutral dots, hover fills, card outlines and off-scale spacing on tokens - #1999

Merged
Makisuo merged 1 commit into
mainfrom
style/consistency-pass-3
Oct 30, 2026
Merged

Makisuo merged 1 commit into
mainfrom
style/consistency-pass-3

style(web): neutral dots, hover fills, card outlines and off-scale sp…

bff60c1
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 30, 2026 in 54s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. Every token and utility the pass introduces resolves against the shared theme, and no logic, state or data path changes.
no findings · tests not needed · risk low

A visual-only Tailwind pass: neutral fills, dividers, hover fills, radii, gaps and name-column widths move onto the existing scales, plus one new --color-border-strong token (foreground 30%). No logic, markup structure or data path changes, so it is safe to merge.

  • tokens.css gains --color-border-strong, used for hover and selected card outlines
  • Status dots and rails collapse onto bg-foreground-subtle / bg-foreground-faint
  • Dividers use bg-border-subtle; hover fills use bg-surface-sunken / bg-surface-faint
  • Name-column mins move to the 16px width scale (min-w-44 … min-w-64)
What was checked
  • All referenced tokens exist in packages/ui/src/styles/tokens.css:430-437, imported by web, landing and local-ui
  • Numeric min-w-*/max-w-*/rounded-t-xs are valid Tailwind 4.3 utilities; header and row min-widths still match in each table
  • No class-string equality or lookup depends on the replaced values; conditional branches keep one class each

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