Skip to content

fix(web): make the add-on stepper's total-quantity semantics explicit - #968

Merged
jiashuoz merged 1 commit into
mainfrom
fix/addon-total-semantics
Aug 29, 2026
Merged

fix(web): make the add-on stepper's total-quantity semantics explicit#968
jiashuoz merged 1 commit into
mainfrom
fix/addon-total-semantics

Conversation

@jiashuoz

Copy link
Copy Markdown
Member

Found dogfooding #960/#967 on staging: owning 1 add-on and stepping to 2 showed Update add-ons + New total: $4/mo, which reads as "buy 2 more for another $4" — the stepper is the account total, but nothing said so.

  • Visible Total add-ons label on the stepper.
  • Staged changes name the total on the button: Update to 2 add-ons / Buy 2 add-ons.
  • The preview states total and delta: New monthly total: $4/mo (+1 add-on, +$2/mo) — decreases show negative deltas.

Tests: 2 new cases pinning the reported scenario; suite 17/17, full web green, tsc + prod build clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ScEpytuD7GvaXEssvJ2W32

Dogfooding on staging surfaced the misread immediately: owning 1
add-on and stepping to 2, the page showed 'Update add-ons' with 'New
total: $4/mo' — which reads as 'buy 2 more for another $4', when it
means 'own 2 in total, +$2/mo'. On a money control that ambiguity is
a defect.

- The stepper gains a visible 'Total add-ons' label.
- A staged change names the total on the button itself: 'Update to 2
  add-ons' / 'Buy 2 add-ons'.
- The preview states both the new monthly total AND the delta this
  click commits: 'New monthly total: $4/mo (+1 add-on, +$2/mo)'
  (negative deltas rendered on decreases).

Tests: 2 new cases pinning the reported scenario (add one more to an
owned add-on; reduce shows a negative delta); labels updated across
the suite, 17/17; full web suite green; tsc + prod build clean.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ScEpytuD7GvaXEssvJ2W32
@jiashuoz
jiashuoz merged commit c6cb914 into main Aug 29, 2026
29 checks passed
@jiashuoz
jiashuoz deleted the fix/addon-total-semantics branch August 29, 2026 22:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant