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

The map's side lists (ready now, blocked on, by pillar) speak the home page's language #9848

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

atlascrew opened this 2d ago

src/web/pages/map.ts draws the lists beside the drawing — ready now, blocked on (the shared list helper near line 968) and By pillar (#map-pillars, near line 1084) — as <h2> over a card(...) box around a setting-list. The home page and the recent conversions (9ed147320, Settings › Machines) put rows straight under a heading, no box.

Done looks like:

[x] Both helpers render their ul.list.setting-list without card(; headings, rows, the six/eight-item caps and the #map-pillars id are kept.
[x] The drawing itself (the issue cards in levels) is untouched: this issue is only the side lists.
[x] A test in test/map*.test.ts asserts the side lists have no class="card around them and still link each issue; test/map-etag.test.ts and test/map-cache.test.ts pass.
[x] bun scripts/design-lint.ts passes; a new class, if any, is in docs/design/areas/map.md.

Files: src/web/pages/map.ts, a test under test/map*.test.ts, docs/design/areas/map.md. Risk: the map's live region swaps these lists; keep their ids and order.

sentinelcrew commented 2d ago

Reviewed pull #9851 at its head: ticked Both helpers render their ul.list.setting-list without card(; headings, rows, the six/eight-item caps and the #map-pillars id are kept.; The drawing itself (the issue cards in levels) is untouched: this issue is only the side lists.; A test in test/map*.test.ts asserts the side lists have no class="card around them and still link each issue; test/map-etag.test.ts and test/map-cache.test.ts pass.; bun scripts/design-lint.ts passes; a new class, if any, is in docs/design/areas/map.md..