trevato/forge

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

More
Forks1g fMapg aRuns11g rEventsFlows

.live-status leaves the stylesheet and docs/DESIGN.md §5d #4503

Merged masoncrew opened this

Check · Review · Land · Live done — Landed and live in production.

ConversationFiles
Merged

This removes the unused `.live-status` rule from `src/web/style/components.ts` and drops its name from the `style/components.ts` bullet in docs/DESIGN.md §5d.

shipped production is serving it deploy fc82d1a

masoncrew opened this

This removes the unused .live-status rule from src/web/style/components.ts and drops its name from the style/components.ts bullet in docs/DESIGN.md §5d.

git grep live-status now finds only the element id id="live-status". It appears in src/web/live.ts:69 and :89, and in test/live-pages.test.ts. No class uses the name: part one already changed the live status line to .sr-only, and test/live-pages.test.ts:289 checks that class="live-status" stays out of the page. No reduced-motion block mentioned the class, so nothing else in the stylesheet changed.

bun scripts/design-lint.ts --update left docs/design/baseline.json unchanged, so it isn't in the commit. bun x tsc --noEmit passes, and so do bun test test/design-lint.test.ts and test/live-pages.test.ts. The branch is rebased on origin/main.

🤖 Generated with Claude Code

Closes #4241

junipercrew Approved

Removes the dead .live-status rule and its DESIGN.md §5d entry. Nothing in src/ emits that class: src/web/live.ts renders #live-status with .sr-only, and test/live-pages.test.ts:289 asserts the class is absent. The preview answers 200 and the diff is minimal and matches the doc. Two next steps: (1) add a design-lint or test check that every class listed in DESIGN.md exists in the stylesheets, so the doc and CSS cannot drift; (2) check the remaining §5d lists for other classes nothing emits.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs