From 60aa8a888e8c9037cab3680f07fd535feccde44c Mon Sep 17 00:00:00 2001 From: Lilleman auf Larv Date: Wed, 9 Sep 2026 15:16:36 +0200 Subject: [PATCH] Point the contract doc at a heading that exists, and let the table say which axes it scrolls --- e2e-tests/visual.spec.ts | 7 ++----- public/css/styles.css | 2 +- views/partials/data-table.ejs | 2 +- 3 files changed, 4 insertions(+), 7 deletions(-) diff --git a/e2e-tests/visual.spec.ts b/e2e-tests/visual.spec.ts index 558b0f1..2cd9cf9 100644 --- a/e2e-tests/visual.spec.ts +++ b/e2e-tests/visual.spec.ts @@ -28,11 +28,8 @@ test.beforeEach(async ({ context }) => { await context.addCookies([{ name: SESSION_COOKIE, url: BASE_URL, value: devSession() }]); }); -// The shell must never clip a page: a body that does not scroll itself has to reach the reader -// through the document. One page per body idiom, since the change removed a bounded rule from each -// (.form-page never had one, .shell-auth did). A key press, not scrollIntoView — a script can scroll -// an overflow-hidden box, a reader cannot; and not the wheel, which Firefox's synthetic event never -// delivers to the document. +// A key press, not scrollIntoView — a script can scroll an overflow-hidden box, a reader cannot; +// and not the wheel, which Firefox's synthetic event never delivers to the document. for (const [name, path, tail] of [ ["the starter dashboard", "/dashboard", ".form-actions .btn"], ["the public landing", "/", ".landing-actions .btn"], diff --git a/public/css/styles.css b/public/css/styles.css index 6461f71..aba6f76 100644 --- a/public/css/styles.css +++ b/public/css/styles.css @@ -562,7 +562,7 @@ span.nav-self { cursor: default; } /* static / non-clickable */ /* ---------- 9. TABLE --------------------------------------- */ /* A bounded region is opt-in: give this a height and the header below stays put inside it. */ -.table-wrap { overflow-x: auto; } +.table-wrap { overflow: auto; } table.table { width: 100%; border-collapse: separate; border-spacing: 0; font-size: var(--fz); font-variant-numeric: tabular-nums; diff --git a/views/partials/data-table.ejs b/views/partials/data-table.ejs index f8e2c08..04e1227 100644 --- a/views/partials/data-table.ejs +++ b/views/partials/data-table.ejs @@ -7,7 +7,7 @@ name it when two tables share a page scrollRegion? take the leftover height and scroll, so the header stays put — needs the shell's `fill: true` and a flex chain - above (README → the app shell) + above (README → The menu system) columns: { label, sortable?, sort?: "asc"|"desc", href?, className? }[] rows: { name?, cells: Cell[], actions?: Action[] }[] Cell ∈ string | { text, className? } | { user:{name,initials} } | { rowHeader:{text,href?} } | { badge:{tone,label} } | { html, className? }