Skip to content

style(web): polish onboarding from the first screen to the top bar checklist - #2015

Merged
Makisuo merged 4 commits into
mainfrom
style/onboarding-polish
Oct 31, 2026
Merged

Makisuo merged 4 commits into
mainfrom
style/onboarding-polish

Merge remote-tracking branch 'origin/main' into style/onboarding-polish

4c00d6c
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Oct 31, 2026 in 1m 15s

Confidence 10/10 · No issues found

🟢 Confidence 10/10 · safe to merge. Copy and class changes only, pinned by updated scene tests; the one behavior change (role="alert") is strictly stronger than the removed aria-live.
no findings · tests covered · risk low

A visual polish pass over every onboarding screen, the top-bar credits checklist and the overview setup card, plus two lab preview pages. Only classes, copy and one exported panel function change, apart from the create-org error now using role="alert"; safe to merge.

  • optionCard drops accent/checkClassName; selected cards always use border-primary
  • Live-step cards become one link each, Back moved beside "Go to {org}"
  • checklistPanel exported from shell/onboarding-checklist-view.ts for the lab
  • New /lab/onboarding-checklist and ?step=org|region lab routes
What was checked
  • No caller passes the removed optionCard options; checkClassName has no matches repo-wide
  • Region and plan steps stay scrollable through main's overflow-y-auto (quick-start-view.ts:196)
  • Lab ?step=org|region reaches the region screen via needsRegion (quick-start-view.ts:1242)

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