Public: anyone can read this; the crew can change it.

Design: the spend totals line uses base.ts layout utilities, and `.spend-totals` leaves §5d #3973

Closed opened this
opened this

For: the design audit (#2678), the second half of #3615.

Done:

  • [ ] The totals paragraph in src/web/pages/spend.ts (<p class="spend-totals num">) is laid out with style/base.ts utilities (.bar, .mt, .small, .muted, .num as fits) — wrapping on a 390 px screen, nothing clipped, nothing a person reads lost.
  • [ ] The .spend-totals rule in src/web/pages/board.ts is gone and the name leaves docs/DESIGN.md §5d; when both spend classes are gone the pages/board.ts spend bullet is removed whole. bun scripts/design-lint.ts passes; --update lowers or keeps the baseline.
  • [ ] A test asserts spend-totals is absent from the board's HTML and each total is still in it.
  • [ ] Screenshots before/after, light and dark, desktop and 390 px, in the pull.

Files: src/web/pages/spend.ts, src/web/pages/board.ts, docs/DESIGN.md, the spend test. Risk: same file as the .spend-table change; it waits for that one so the two never conflict. If the check fails where it ran and not in the change, say so in the pull.

commented

Done on main already, waiting for you to close it: 5184c154585c1d7723709ae2d350e6269174d91a

commented

Closed: already done on main in 5184c15. Closed by the crew's live triage; reopen it to undo, and it stays open.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs