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

/map nodes are named by aria-label, not an SVG <title>, and all three tests that read the <title> read the aria-label #3229

shipped production is serving it opened this has it
opened this

Replaces #3223. Its two tries (pulls #3225, #3226) failed the same way. They changed src/web/pages/map.ts and test/limbs.test.ts, but test/plan-pillars.test.ts:257 ("a plan says what each issue is for > the map page draws the deferred issue deferred, with why, and the pillar issue ready") still expects — deferred: behind the Later gate</title>.

Done looks like:

  • [ ] In src/web/pages/map.ts (the <a class="map-node"> near line 678), the <title>…</title> child is removed. The same text goes in an aria-label="…" on the <a>: #N title — word[: by who | : why], escaped through html exactly as before.
  • [ ] test/plan-pillars.test.ts (~line 256) reads the node's aria-label and expects it to end with — deferred: behind the Later gate. The test still fails if the reason is missing.
  • [ ] test/limbs.test.ts (~line 453) expects aria-label="#${extra.number} One home: the footer — deferred: ${through}" in place of the <title> text.
  • [ ] "title /map" is removed from KNOWN_SIGNED_IN in test/forge-webspec.test.ts (line 44), and that suite passes.
  • [ ] Before pushing, grep -rn "</title>" test/ | grep -v studio-words returns no line about the map. Then bun test test/plan-pillars.test.ts test/limbs.test.ts test/forge-webspec.test.ts test/design-map.test.ts test/map-cache.test.ts passes.

Files: src/web/pages/map.ts, test/plan-pillars.test.ts, test/limbs.test.ts, test/forge-webspec.test.ts. Risk: the card regex in plan-pillars expects the title in the node's visible text after the opening tag. That still holds, because the aria-label sits inside the tag.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs