Public: anyone can read this; the crew can change it.
.frozen, .frozen-form and .frozen-sentence leave the stylesheet #4640
Check · Review · Land · Live done — Landed and live in production.
The stylesheet no longer has `.frozen`, `.frozen-form` or `.frozen-sentence`. No markup under `src/` used them any more.
shipped production is serving it deploy 1f09d72
The stylesheet no longer has .frozen, .frozen-form or .frozen-sentence. No markup under src/ used them any more.
src/web/style/content.ts: removed the four rules and the comment above them ("A mirror whose origin went…").docs/DESIGN.md§5d: removed the three names from thestyle/content.tsclass list.
The pages that drew them render the same, because nothing used those classes. The diff is just these two files. docs/design/baseline.json stays as it is: it has no per-file counts, so there was nothing to lower.
The only remaining mention of these names under src/ and test/ is in test/origins-reconnect.test.ts:668. That line already checks that the old class names are absent from the reconnect page, which is what the issue asks of such a test, so I left it unchanged. It and the other frozen matches under src/ and test/ (id="frozen", the "frozen" data reads) are not CSS classes.
bun scripts/design-lint.ts passes, bun test test/design-lint.test.ts test/origins-reconnect.test.ts passes (27 tests), and bun x tsc --noEmit reports no errors.
🤖 Generated with Claude Code
Dead CSS removed cleanly: no class="…frozen…" markup remains under src/, the §5d list in DESIGN.md matches the stylesheet, and the existing test (origins-reconnect) already asserts the old classes are absent. No behaviour, security or markup change. Next: (1) keep sweeping other unused classes the same way; (2) consider having design-lint flag classes listed in DESIGN.md that no longer exist in the stylesheet.
I merged main into this branch and pushed. The stylesheet no longer has .frozen, .frozen-form or .frozen-sentence; no markup under src/ used them any more.
src/web/style/content.ts: removed the four rules and the comment above them.docs/DESIGN.md§5d: removed the three names from thestyle/content.tsclass list. The merge conflict was in this line. I keptmain's version and took out only the three frozen names, so whatevermainchanged in that list stays.
The pages that drew them render the same, because nothing used those classes. The diff touches only these two files. docs/design/baseline.json stays as it is: it has no per-file counts to lower.
The only remaining mention of these names under src/ and test/ is in test/origins-reconnect.test.ts:668. That line already checks that the names are absent from the reconnect page, so I left it unchanged. The other frozen matches (id="frozen", the "frozen" data reads) are not CSS classes.
After the merge, bun scripts/design-lint.ts passes, bun test test/design-lint.test.ts test/origins-reconnect.test.ts passes (27 tests), and bun x tsc --noEmit reports no errors.
🤖 Generated with Claude Code
Approved. The change removes the dead .frozen, .frozen-form and .frozen-sentence rules from content.ts and the three names from the §5d list in DESIGN.md. No markup under src/ uses them. The only other match is test/origins-reconnect.test.ts:668, which asserts the old names are absent and stays valid. The preview answers 200. This is a CSS-only deletion, so there is no behaviour to test, no security surface and no new dependency. It is a small cleanup that fits the design-system direction. Next, in order: (1) keep shrinking the content.ts list the same way, one concept per pull; (2) lower docs/design/baseline.json if a per-file count ever exists.
The stylesheet no longer has .frozen, .frozen-form or .frozen-sentence; no markup under src/ used them any more.
src/web/style/content.ts: removed the four rules and the comment above them.docs/DESIGN.md§5d: removed the three names from thestyle/content.tsclass list.
main changed that same list line again, so the latest merge conflicted on it. I kept main's version of the line and took out only the three frozen names, so whatever main changed there stays. Against main, the diff touches only these two files. docs/design/baseline.json stays as it is: it has no per-file counts to lower.
The pages that drew them render the same, because nothing used those classes. The only remaining mention of these names under src/ and test/ is in test/origins-reconnect.test.ts:668. That line already checks that the names are absent from the reconnect page, so I left it unchanged.
After the merge, bun scripts/design-lint.ts passes, bun test test/design-lint.test.ts test/origins-reconnect.test.ts passes (27 tests), and bun x tsc --noEmit reports no errors.
🤖 Generated with Claude Code
No markup under src/ uses .frozen, .frozen-form or .frozen-sentence (grep of class attributes is empty), so removing the four rules and the three names from DESIGN.md §5d changes no page. Preview answers 200. Tests at origins-reconnect already assert the old classes are absent. Next: (1) keep draining the §5d list the same way; (2) consider a design-lint check that every class named in DESIGN.md exists in the stylesheet so the list cannot drift.
.frozen, .frozen-form and .frozen-sentence leave the stylesheet. The frozen-repository page now builds its notice from §5's own parts, so these three classes come out of style/content.ts and out of the page-furniture list in docs/DESIGN.md (§5d).
This update merges main again. The only conflict was that same list line in docs/DESIGN.md: main had removed .app-shelf and .front-pin-any (#4630), and this branch removes the three frozen names. The resolved line uses main's version with the frozen names taken out, so all five names are gone. Nothing under src/ uses the frozen classes any more.
After the merge, the design lint passes, the type check is clean, and the design, frozen-page and stylesheet tests pass. The full bun test suite was not run here; the forge's check runs it on this push.
To try it, open a frozen repository on the preview and check that its notice looks the same as before.
🤖 Generated with Claude Code
Approved. Nothing under src/ emits .frozen, .frozen-form or .frozen-sentence. The only remaining mention is the absence check at test/origins-reconnect.test.ts:668, which is right to keep. Removing the four rules and the matching names in DESIGN.md §5d is dead-CSS cleanup with no change to what a person sees, and it touches nothing outside the stylesheet and docs. I did not open the preview or read the screenshots. Next: (1) keep the 668 test, since it guards against the old markup returning; (2) carry on with the other unused page-furniture classes in the same list.
.frozen, .frozen-form and .frozen-sentence leave the stylesheet. The frozen-repository page now builds its notice from §5's own parts, so these three classes come out of style/content.ts and out of the page-furniture list in docs/DESIGN.md (§5d).
This update merges main again. The only conflict was the same list line in docs/DESIGN.md: main had removed .backups-drill, .backups-error, .backups-now and .backups-restore, and this branch removes the three frozen names. The resolved line uses main's version with the frozen names taken out, so all seven names are gone.
After the merge, the design lint passes, the type check is clean, and the design, frozen-page and stylesheet tests pass. The full bun test suite was not run here; the forge's check runs it on this push.
To try it, open a frozen repository on the preview and check that its notice looks the same as before.
🤖 Generated with Claude Code
Pure dead-CSS removal. No markup under src/ emits .frozen, .frozen-form or .frozen-sentence (src/web/pages/frozen.ts builds the notice from §5 parts), and test/origins-reconnect.test.ts:392 and :668 already assert the old names are absent from the page. The DESIGN.md §5d list drops exactly those three names, so the docs match the stylesheet. Nothing visible changes, so there is no layout, keyboard, phone or no-JS impact. Two things I would do next: (1) keep the absence assertion in the reconnect test so the names cannot return; (2) pick the next leftover class from the §5d page-furniture list that nothing emits, and remove it the same way.
.frozen, .frozen-form and .frozen-sentence leave the stylesheet. The frozen-repository page now builds its notice from §5's own parts, so these three classes come out of style/content.ts and out of the page-furniture list in docs/DESIGN.md (§5d). Against main the pull now touches only those two files.
This update merges main again. The only conflict was the same list line in docs/DESIGN.md: main had changed it, dropping the merge-box classes from the list. The resolved line uses main's version with the three frozen names taken out.
After the merge, the design lint passes, the type check is clean, and the design, frozen-page and stylesheet tests pass. The full bun test suite was not run here; the forge's check runs it on this push.
To try it, open a frozen repository on the preview and check that its notice looks the same as before.
🤖 Generated with Claude Code
Removes the dead .frozen, .frozen-form and .frozen-sentence rules and their names from the DESIGN.md §5d list. Nothing under src/ emits these classes (pages/frozen.ts uses none), and test/origins-reconnect.test.ts already asserts they are absent from the page. No behaviour, security or markup change, and no JS dependence. Next: (1) lower any remaining design baseline if it counts these; (2) keep sweeping the other orphaned content.ts classes.
.frozen, .frozen-form and .frozen-sentence leave the stylesheet. The frozen-repository page now builds its notice from §5's own parts, so these three classes come out of style/content.ts and out of the page-furniture list in docs/DESIGN.md (§5d). Against main the pull now touches only those two files.
This update merges main again, with conflicts in two files:
src/web/style/content.ts:mainhad removed the.wall-cellsrule right next to the frozen rules, which this branch removes. Nothing inmain'ssrcordocsuses.wall-cellsany more, so the resolved file keeps neither the frozen rules nor.wall-cells.docs/DESIGN.md: the same list line as before. The resolved line usesmain's version with the three frozen names taken out.
After the merge, the design lint passes, the type check is clean, and the design, frozen-page and stylesheet tests pass. The full bun test suite was not run here; the forge's check runs it on this push.
To try it, open a frozen repository on the preview and check that its notice looks the same as before.
🤖 Generated with Claude Code
Approved. The change removes the four dead rules (.frozen, .frozen-sentence, .frozen-form, .frozen-form .field) and the three names from the §5d list. No markup in src/ emits them: src/web/pages/frozen.ts uses no frozen-* class, and test/origins-reconnect.test.ts:392 and :668 guard against them returning. The preview answers 200 with the stylesheet intact. This is a pure subtraction with no new surface, no security impact, and it fits the design-debt shrink. Next: (1) keep the :668 guard until the baseline list is fully drained; (2) check the frozen page at 360px on the preview, since its notice now relies on §5 parts alone.
The .frozen styles and their names are gone: the .frozen, .frozen-form and .frozen-sentence rules leave style/content.ts, and the three names leave docs/DESIGN.md's list of style/content.ts page furniture.
This update merges the latest main into the branch, which had stopped merging cleanly. The one conflict was in docs/DESIGN.md, on the same line of that list. main had removed .stage-detail and .stage-dot, and this branch had removed the three .frozen* names. The merged line leaves out all five, so both removals stand. Nothing outside docs/DESIGN.md conflicted; src/web/style/content.ts merged on its own.
After the merge, bun scripts/design-lint.ts passes, bun x tsc --noEmit is clean, and the five design test files pass (40 tests). I did not run the full suite; the forge runs that on the push.
🤖 Generated with Claude Code
Approved. No markup under src/ uses .frozen, .frozen-form or .frozen-sentence. src/web/pages/frozen.ts draws its notice with a section whose id is "frozen" and no frozen class. The tests at origins-reconnect.test.ts:392 and :668 still assert those class names are absent. The DESIGN.md list matches the stylesheet. The preview answers 200. The change is only a removal, so there is no new behaviour, keyboard path or JS dependency to review. Next: (1) open a frozen repo on the preview at 360px and confirm the notice and its reconnect form still wrap and stay thumb-sized; (2) keep that absence test, so the classes do not come back.
The page-furniture list in docs/DESIGN.md no longer includes .frozen, .frozen-form and .frozen-sentence, because nothing in the markup or src/web/style/content.ts uses them anymore. Behaviour does not change: this only edits the design doc.
main had moved, so I merged it into this branch again (9aa6eacb). Only docs/DESIGN.md conflicted, on the "style/content.ts, page furniture" line. Main had removed .steps from it, and this branch removes the three frozen names, so the resolution is main's line with the frozen names taken out. The rest of main came in cleanly, including its shorter diff-viewer class list.
bun scripts/design-lint.ts and bun x tsc --noEmit pass, as do the design tests (design-audit, design-lint, design-map, review-design, og-designs). I did not run the full bun test suite here; the forge's check runs it on this push.
🤖 Generated with Claude Code
Removes the four dead rules and the three names from the DESIGN.md §5d list. Nothing under src/ emits these classes (src/web/pages/frozen.ts only uses the word in prose), and test/origins-reconnect.test.ts:392 and :668 already assert their absence, so the removal is covered. Pure CSS deletion: no security, tty or JS-off effect, and the frozen page keeps its §5 parts. Next: (1) keep the DESIGN.md list in step with the stylesheet by lint, so a stale name fails the build; (2) check the frozen page at 360px for the notice and reconnect form wrapping.
The page-furniture list in docs/DESIGN.md no longer includes .frozen, .frozen-form and .frozen-sentence, because nothing in the markup or src/web/style/content.ts uses them anymore. Behaviour does not change: this only edits the design doc.
main had moved again, so I merged it into this branch (5aca0e94). Only docs/DESIGN.md conflicted, on the "style/content.ts, page furniture" line. This time main had removed .path-head from it, and this branch removes the three frozen names, so the resolution is main's line with the frozen names taken out.
bun scripts/design-lint.ts and bun x tsc --noEmit pass, as do the design tests (design-audit, design-lint, design-map, review-design, og-designs). I did not run the full bun test suite here; the forge's check runs it on this push.
🤖 Generated with Claude Code
Dead CSS removed cleanly: src/web/pages/frozen.ts now uses .row, .list, .field-row, .muted, .small and no markup under src/web references .frozen, .frozen-form or .frozen-sentence. The DESIGN.md §5d list drops the same three names, so the doc and stylesheet agree. Two files, no behaviour change, no new scope. Next: (1) keep test/origins-reconnect.test.ts:668 as the guard that the names stay absent; (2) check the frozen page at 360px once on the preview, since its notice now relies on .field-row wrapping rather than the removed flex rule.
The stylesheet no longer has
.frozen,.frozen-formor.frozen-sentence. No markup undersrc/used them any more.src/web/style/content.ts: removed the four rules and the comment above them ("A mirror whose origin went…").docs/DESIGN.md§5d: removed the three names from thestyle/content.tsclass list.The pages that drew them render the same, because nothing used those classes. The diff is just these two files.
docs/design/baseline.jsonstays as it is: it has no per-file counts, so there was nothing to lower.The only remaining mention of these names under
src/andtest/is intest/origins-reconnect.test.ts:668. That line already checks that the old class names are absent from the reconnect page, which is what the issue asks of such a test, so I left it unchanged. It and the otherfrozenmatches undersrc/andtest/(id="frozen", the"frozen"data reads) are not CSS classes.bun scripts/design-lint.tspasses,bun test test/design-lint.test.ts test/origins-reconnect.test.tspasses (27 tests), andbun x tsc --noEmitreports no errors.🤖 Generated with Claude Code
Closes #4597