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

The scrolling table is named in docs/design/areas/table.md, and .table-scroll leaves §5d #5549

Merged opened this

Check · Review · Land · Live done — Landed and live in production.

Merged

`.table-scroll` is now documented in its own area file, `docs/design/areas/table.md`, and its line is gone from §5d in `docs/DESIGN.md`. This is a documentation-only change: no markup or stylesheet rule changed, so every page looks the same.

shipped production is serving it deploy 28f939d

opened this

.table-scroll is now documented in its own area file, docs/design/areas/table.md, and its line is gone from §5d in docs/DESIGN.md. This is a documentation-only change: no markup or stylesheet rule changed, so every page looks the same.

  • docs/design/areas/table.md (new): says what the component is for: a horizontal scroller around a wide §5 .table, so only the table's figures move sideways and the page never does. It names the rule's stylesheet, src/web/style/components.ts (overflow-x: auto, thin scrollbar), and points to .table[data-sticky-lead] for keeping each row's name in view. It also lists everything that writes the class: eleven pages under src/web/pages/, the figures table in fleetGraphSection (src/web/fleet-graph.ts), and vitalsTable (src/web/studio/vitals.ts). The format follows docs/design/areas/README.md.
  • docs/DESIGN.md: the one ` - .table-scroll ` line is deleted from §5d, and nothing else.

The class still has both a rule and markup, so nothing needs a separate retirement issue. bun scripts/design-lint.ts passes, and so does test/design-lint.test.ts (16 tests).

To try it: run bun scripts/design-lint.ts and bun test test/design-lint.test.ts.

🤖 Generated with Claude Code

Closes #5527

commented

.table-scroll is now documented in its own area file, docs/design/areas/table.md, and its line is gone from §5d in docs/DESIGN.md. This is a documentation-only change: no markup or stylesheet rule changed, so every page looks the same.

  • docs/design/areas/table.md (new): says what the component is for: a horizontal scroller around a wide §5 .table, so only the table's figures move sideways and the page never does. It names the rule's stylesheet, src/web/style/components.ts (overflow-x: auto, thin scrollbar), and points to .table[data-sticky-lead] for keeping each row's name in view. It also lists everything that writes the class: eleven pages under src/web/pages/, the figures table in fleetGraphSection (src/web/fleet-graph.ts), and vitalsTable (src/web/studio/vitals.ts). The format follows docs/design/areas/README.md.
  • docs/DESIGN.md: the one ` - .table-scroll ` line is deleted from §5d, and nothing else.

The class still has both a rule and markup, so nothing needs a separate retirement issue. bun scripts/design-lint.ts passes, and so does test/design-lint.test.ts (16 tests).

To try it: run bun scripts/design-lint.ts and bun test test/design-lint.test.ts.

🤖 Generated with Claude Code

Approved — before the latest push

Docs-only and accurate: the writer list in table.md matches every src file that emits table-scroll (11 pages, fleet-graph.ts, studio/vitals.ts), the rule is in style/components.ts, and the DESIGN.md §5d line is removed with nothing else touched. No page, markup or stylesheet changes, so nothing to regress on a phone, keyboard or screen reader. Next: (1) keep the writer list from drifting by letting design-lint check that each named file still contains the class; (2) mention the sticky-lead pairing with a one-line example of the markup so the next page author gets it right.

commented

.table-scroll is now described only in docs/design/areas/table.md, which says what it does and which pages use it. Its entry in the unnamed-classes list in §5d of docs/DESIGN.md is gone.

This push merges main back in. The only conflict was in that same §5d list: this branch removed .table-scroll from it, and main removed the line next to it, .tell-here, which now lives in docs/design/areas/tell.md. The resolution keeps both removals, so neither class appears in docs/DESIGN.md. Each is described once, in its own area file. The design-audit, design-lint and design-map tests pass (25 tests); I didn't run the full suite.

To try it, read docs/design/areas/table.md and check that .table-scroll no longer appears in docs/DESIGN.md.

🤖 Generated with Claude Code

Approved

Docs-only and accurate. I checked the rule at src/web/style/components.ts:500 and the data-sticky-lead rule beside it. The 13 writers listed in table.md match exactly what grep table-scroll src finds. design-lint passes. Nothing in the runtime changes, so the preview is unchanged and there is no security, tty or test surface to cover. Two things I would do next: (1) add a short test or lint rule that fails when an area file names a source file that no longer writes its class, so this list cannot rot. (2) Keep going down the §5d unnamed list one class per pull.

Sign in to join this conversation.

↑↓ navigate ↵ open ⌘↵ its app esc close

Keyboard shortcuts

Docs