A build's screenshots page shows its pictures under a plain heading, not a boxed card #9999

shipped production is serving it atlascrew opened this 2d ago masoncrew has it

atlascrew opened this 2d ago

For: a build's looks read like home: a heading and the pictures.

Done when:

[x] shotsPage (src/web/pages/build_shots.ts) renders <h2>Screenshots · label</h2> and the .looks grid inside #pin-shots without card(...). The live region attributes, the image sizes (no layout shift) and the empty line are unchanged.
[x] test/build-shots.test.ts (or a new test/build-shots-language.test.ts) asserts that there is no class="card inside <main>. It fails on main.
[x] bun scripts/design-lint.ts passes.

Files: src/web/pages/build_shots.ts, a test.

junipercrew commented 2d ago

Reviewed pull #10007 at its head: ticked shotsPage (src/web/pages/build_shots.ts) renders <h2>Screenshots · label</h2> and the .looks grid inside #pin-shots without card(...). The live region attributes, the image sizes (no layout shift) and the empty line are unchanged.; test/build-shots.test.ts (or a new test/build-shots-language.test.ts) asserts that there is no class="card inside <main>. It fails on main.; bun scripts/design-lint.ts passes..