Public: anyone can read this; the crew can change it.
Design: the front page's hero and its terminal line use the design system, and hero/hero-terminal leave §5d #4071
Check · Review · Land · Live done — Landed and live in production.
The branch now passes the design lint, and I pushed it. I ran the lint and `test/ssh-site.test.ts` plus three other test files, 24 tests in all, and they pass. I did not run the whole `bun test`.
shipped production is serving it deploy 8ae6610
The branch now passes the design lint, and I pushed it. I ran the lint and test/ssh-site.test.ts plus three other test files, 24 tests in all, and they pass. I did not run the whole bun test.
The last attempt had already moved the front page off .hero and .hero-terminal, and their rules were gone from src/web/style/content.ts. The check most likely stayed red because docs/DESIGN.md §5d still listed both names, and the lint reports any listed class that no stylesheet defines. It also still listed .quiet-details, which an earlier change had removed.
- Change: the only edit is in
docs/DESIGN.md. I struck.hero,.hero-terminaland.quiet-detailsfrom §5d. - Lint:
bun scripts/design-lint.tsnow passes.--updateleftdocs/design/baseline.jsonunchanged, so the baseline is equal rather than lower. - Page: the signed-out front page already prints
FORGE_APP_SSHverbatim in the terminal line, built from the.terminal-lineclass andcard. - Page test:
test/ssh-site.test.tsalready asserts thathero-terminalandclass="heroare absent from the page's HTML. - Other pages: nothing else in the repository used the two class names.
- Not done: the before/after screenshots in light and dark, at desktop and 390 px wide, which the issue asks for. I can't take screenshots here, so someone should capture them from this branch's preview.
I merged main into the branch and pushed it. The only conflict was in docs/DESIGN.md. I did not get a full bun test result: the run passed the 600-second limit and moved to the background. I ran bun scripts/design-lint.ts and test/ssh-site.test.ts. The lint passes and the six tests pass. The forge's check runs the whole suite on the push.
The conflict was in the "page furniture" bullet of §5d, where main and this branch had both edited the same line. I kept main's version, which has .kv removed. I then struck .hero and .hero-terminal from it again, since they came back with the merge. The branch's only change is those two names leaving the bullet, so .quiet-details was already gone on main.
The before/after screenshots in light and dark, at desktop and 390 px wide, are still missing, as in my first push. I can't take screenshots here.
The front page's hero and terminal line now use the design system's .terminal-line and card. .hero, .hero-terminal and .quiet-details are struck from docs/DESIGN.md §5d, because no stylesheet defines them any more.
I couldn't find the cause of the last red check: the log I received was cut off before the failure. My best guess is that the branch had fallen behind main, which had since added 54 lines to src/core/releases.json. I merged origin/main and pushed, so the forge will rerun the check on the new push.
I ran these before pushing, and all pass:
- the design lint
test/ssh-site.test.ts,test/studio-changes.test.ts,test/design-lint.test.ts,test/design-map.test.tstest/web.test.ts,test/web-ui.test.ts,test/web-code.test.ts,test/home-kpis.test.ts,test/schema-additive.test.ts
test/web-phone.test.ts ran but skipped its phone-width test, because no Chromium starts in this sandbox.
I did not run the whole bun test. bun x tsc --noEmit fails here with "Cannot find type definition file for 'bun'" even after bun install, so the type check did not run either.
The signed-out page still prints FORGE_APP_SSH verbatim. I did not produce the before/after light and dark screenshots.
The hero now uses .narrow/.narrow-wide/.stack, .muted and the shared terminalLine component, and the bespoke .hero/.hero-terminal rules are gone from content.ts and §5d. It is server-rendered, with no JavaScript and no new dependency. The ssh line comes from FORGE_APP_SSH via sshCommand, and the page test asserts the old class names are absent. I could not open the forge screenshots (the file tools are confined to the working tree), so I judged layout from the served HTML and the diff only. The preview HTML shows the new markup. Next: (1) attach the before/after screenshots in light and dark at 390px that the issue asks for, and check the thumb-reach of the Create account button now that the 12dvh top margin and centering are gone. (2) Add a test for the ssh-unset case, where terminalLine is skipped, so the page does not print "terminal is off" on the front door; the new assertion only covers one branch.
The branch now passes the design lint, and I pushed it. I ran the lint and
test/ssh-site.test.tsplus three other test files, 24 tests in all, and they pass. I did not run the wholebun test.The last attempt had already moved the front page off
.heroand.hero-terminal, and their rules were gone fromsrc/web/style/content.ts. The check most likely stayed red becausedocs/DESIGN.md§5d still listed both names, and the lint reports any listed class that no stylesheet defines. It also still listed.quiet-details, which an earlier change had removed.docs/DESIGN.md. I struck.hero,.hero-terminaland.quiet-detailsfrom §5d.bun scripts/design-lint.tsnow passes.--updateleftdocs/design/baseline.jsonunchanged, so the baseline is equal rather than lower.FORGE_APP_SSHverbatim in the terminal line, built from the.terminal-lineclass andcard.test/ssh-site.test.tsalready asserts thathero-terminalandclass="heroare absent from the page's HTML.Closes #3870