Skip to content

feat(ui): finish the move to Geist: numbers, 13px chrome, sentence-case labels - #2565

Merged
Makisuo merged 2 commits into
mainfrom
feat/design-refine-type
Nov 13, 2026
Merged

Makisuo merged 2 commits into
mainfrom
feat/design-refine-type

fix(ui): review follow-ups for the Geist type pass

34823ad
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Nov 13, 2026 in 1m 21s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. Token and kit-style edits only, with every impacted call site read at this head and the earlier size: "lg" sidebar defect now fixed.
no findings · tests not needed · risk medium

Finishes the Geist type pass: drops the global tabular-nums→mono and text-figure→sans rules, moves chrome to 13px text-ui, and takes uppercase off the eyebrow kit while migrating the call sites that relied on it. Contained to tokens, kit styles and label copy; safe to merge.

  • tokens.css no longer forces font-mono on tabular-nums or font-sans on text-figure
  • Kit controls move to sm:text-ui (13px); sidebar rows and breadcrumbs go to text-ui
  • eyebrow overline/label become sentence case at 11px/12px; mono keeps the uppercase tag
  • statFigure defaults to the display face and statRailItem.isCardTitle is gone

Fixed since the last review

  • ✅ F1 · size: "lg" keeps text-xs, so the org switcher row stays 12px beside 13px rows
What was checked
  • Machine text still declares font-mono itself (badge.styles.ts:32, truncated-id.styles.ts:1, log body cells); charts and table figures fall to Geist on purpose
  • Every lowercase literal label handed to eyebrow was migrated (pods→Pods, err→Errors, duration→Duration); only the mono variant keeps one
  • isCardTitle has no callers left at this head (grep), and the service map height follows the 16px shorter top bar (100vh-9rem at service-map/view.ts:251)

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