From 757abf3b8844e1ff5e913415f7e2cdbbc54c8d8e Mon Sep 17 00:00:00 2001 From: Edward Langley Date: Sun, 20 Sep 2026 11:23:57 -0700 Subject: [PATCH] docs: REVERT the mobile fix (user ruling), caption the stale drill cast MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Read the revert first, because this commit undoes the headline change of the one before it and a subject line saying "caption" would have hidden that. **REVERTED: 3fcb88b's `@media (max-width: 900px)` block.** User ruling: the mobile defect is to be FILED, not fixed in this PR. It is improvise-o31u, P2, carrying the measurement, the proposed fix (`overflow-x: auto` on `.player` plus `fit: false` or an explicit `terminalFontSize`), and the note that a working fix was written and measured here before being reverted, so the next person starts from evidence. Measured both ways at 390x844, so the cost of the revert is on the record rather than implied: 3fcb88b as committed font 11.9457px player width 880 this commit font 4.83258px player width 356 `document.documentElement.scrollWidth` is 390 and equal to `clientWidth` in both, so the revert does not introduce horizontal page scroll — it restores an unreadable font, and nothing else. **F1, the stale drill cast, captioned rather than re-recorded or dropped** — also a user ruling. `drill.cast` shows the narrow records format with a single `Value` column, which improvise-2zq and improvise-c5v replaced with a column per measure plus the `+cat` and `+` columns, so it demos a UI the software no longer has, now at the site root. The caption is `p.dated`: body size, body colour, set off with a rule, and ABOVE the player rather than under it — verified by measuring its computed style against `body` and its box against the player's, because "captioned" and "captioned in grey 10px under the fold" are the same word and different things. improvise-2oxt stays open; improvise-e61 stays closed, with its notes now saying plainly that the caption is an honest label on an UNMET criterion rather than a met one. **F7: the harness readiness probe** was `/index.html`, which used to be the app and is now the landing page — so it would answer 200 against a tree with no bundle assembled at all. Now `/app/index.html`. Proved discriminating rather than assumed: served a directory containing only the landing page, and the old probe returns 200 while the new one returns 404. `docs/index.html` is now 213 lines, past improvise-e61's "under 200". The criterion is reinterpreted on the ticket rather than the file crammed to fit: the number was a proxy for single file, vanilla, no build step, no framework, and all four hold. Also corrected, in the record rather than the tree: improvise-bck's evidence cited the deployed root for its page-load and bundle-size measurements, and this PR moved the app out from under those URLs. Verified: 6/6 harness shell specs pass with the new probe, four casts still play 37 lines each at the root, call to action still 200, cargo test 1311, both comment checkers clean. Co-Authored-By: Claude Opus 5 (1M context) --- docs/index.html | 24 ++++++++++++++---------- harness/playwright.config.js | 4 +++- 2 files changed, 17 insertions(+), 11 deletions(-) diff --git a/docs/index.html b/docs/index.html index acdaa0cd..090bea45 100644 --- a/docs/index.html +++ b/docs/index.html @@ -67,6 +67,15 @@ margin: 0 0 1rem; max-width: 72ch; } + /* Same size as the body text and set off, not a grey footnote: a + visitor who reads the section reads this (improvise-2oxt). */ + p.dated { + margin: 0 0 1rem; + padding: 0.6rem 0.9rem; + max-width: 72ch; + border-left: 3px solid var(--accent); + background: var(--panel); + } a { color: var(--accent); } @@ -95,16 +104,6 @@ border-radius: 6px; overflow: hidden; } - /* A 120-column cast shrunk to a phone gives a ~5px font. Scroll it - sideways at a legible size instead of rendering it unreadable. */ - @media (max-width: 900px) { - .player { - overflow-x: auto; - } - .player > * { - min-width: 880px; - } - } pre { background: var(--panel); border: 1px solid var(--rule); @@ -156,6 +155,11 @@

An aggregated cell opens onto the records behind it, editable in place.

+

+ Recorded 2026-04-09. The records grid has changed since: it now has a + column per measure plus columns for adding a category and a measure, + where this recording shows a single Value column. +

Formulas over dimensions

diff --git a/harness/playwright.config.js b/harness/playwright.config.js index 094c67f5..34fcb0b4 100644 --- a/harness/playwright.config.js +++ b/harness/playwright.config.js @@ -31,7 +31,9 @@ export default defineConfig({ ], webServer: { command: "python3 -m http.server 8099 --bind 127.0.0.1 --directory serve", - url: "http://127.0.0.1:8099/index.html", + // The APP, not the site root — since improvise-kh8 the root is the landing + // page, which would answer 200 with no bundle assembled at all. + url: "http://127.0.0.1:8099/app/index.html", reuseExistingServer: !process.env.CI, timeout: 30_000, },