Skip to content

feat(ui): one answer trail for every step flow - #2576

Open
Makisuo wants to merge 3 commits into
mainfrom
feat/wizard-answer-trail
Open

Makisuo wants to merge 3 commits into
mainfrom
feat/wizard-answer-trail

fix(ui): a long answer truncates inside its trail chip

67b267c
Select commit
Loading
Failed to load commit list.
Maple Review Bot / Maple / review succeeded Nov 13, 2026 in 51s

Confidence 9/10 · No issues found

🟢 Confidence 9/10 · safe to merge. Contained CSS fix in one kit view; the shrink chain that makes the ellipsis appear is verified by reading the classes, and the truncation itself has no test.
no findings · tests partial −1 · risk medium

Replaces five hand-rolled step trails with one kit answerTrail; the last commit lets a long answer ellipsize inside its receipt chip instead of pushing the trail past a phone's width. Safe to merge.

  • answerTrail replaces stepTrail and the hand-rolled trails in connect pages, onboarding, GCP and status page dialogs
  • A done chip renders its answer in a truncate span, with the check icon shrink-0
  • maple/one-step-trail confines aria-current="step" to the kit
What was checked
  • Shrink chain for the ellipsis: min-w-0 + max-w-full on both chip and li bound the span (answer-trail.ts:28,58)
  • icon applies className through ICON_CLASS_MARK, so shrink-0 reaches the check svg (icon.ts:38)
  • pressable merges className via cn and keeps the chip's aria-label (pressable.ts:37,51)

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