Prototype: the drawings follow the app's wording, "Approve…" and "Sweep the shelf…" (D276) #1067

Merged
claude-bot merged 3 commits from docs/prototype-wording-d276 into main 2026-10-04 04:14:26 +00:00
Collaborator

The owner's answer to Q148 (D276): where the app's wording differs from the older drawings, the app is right. Prototype and ledger only; no app code.

Summary

  • Lane 3, Review. The approve button is drawn "Approve…", as ReviewRecording.jsx has it, in place of "Approve the transcript". The summary's approve button, the same family (ReviewSummary.jsx also says "Approve…"), follows in place of "Approve the summary". The confirmation dialogs' own titles are left as drawn.
  • Lane 4, the table. "Sweep the shelf…" replaces "End of session…" on the desktop bar and in the phone foot (#t1phone, #t1side).
  • Ledger and coverage. IS-476 and TB-009's drawn-label note in coverage/lane4.py, and the IS-476 row in ledger-IS.md, say the drawing now matches. CATALOGUE.md is rebuilt from the sources; one row changes.
  • Left alone: the D95 and D276 decision rows, which quote the old words on purpose, and the ledger rows that describe the old app.

Catalogue check. Before any edit, a rebuild from main's sources matched main's committed CATALOGUE.md byte for byte. The textual merges of the last day left no drift.

Changelog

  • No fragment: prototype-only, as with #1047.

Test plan

  • All 16 coverage scripts exit 0, "unclassified: none".
  • Feedback-site tests: 127 passed.
  • Rendered with Playwright in both themes:
    • lane 3 Review ("Approve…" under the Discord tally);
    • lane 4's bar ("Sweep the shelf…" fits on one line);
    • the phone foot;
    • the sideways phone.
  • To look at: in the upright phone drawing, "Sweep the shelf…" wraps to two lines in its button. The app does the same at 390 px, so the drawing is faithful.
  • Not opened: about half of the rendered variants. The summary button's change was made after the render.

🤖 Generated with Claude Code

The owner's answer to Q148 (D276): where the app's wording differs from the older drawings, the app is right. Prototype and ledger only; no app code. ## Summary - **Lane 3, Review.** The approve button is drawn "Approve…", as `ReviewRecording.jsx` has it, in place of "Approve the transcript". The summary's approve button, the same family (`ReviewSummary.jsx` also says "Approve…"), follows in place of "Approve the summary". The confirmation dialogs' own titles are left as drawn. - **Lane 4, the table.** "Sweep the shelf…" replaces "End of session…" on the desktop bar and in the phone foot (`#t1phone`, `#t1side`). - **Ledger and coverage.** IS-476 and TB-009's drawn-label note in `coverage/lane4.py`, and the IS-476 row in `ledger-IS.md`, say the drawing now matches. `CATALOGUE.md` is rebuilt from the sources; one row changes. - **Left alone:** the D95 and D276 decision rows, which quote the old words on purpose, and the ledger rows that describe the old app. **Catalogue check.** Before any edit, a rebuild from main's sources matched main's committed `CATALOGUE.md` byte for byte. The textual merges of the last day left no drift. ## Changelog - [x] No fragment: prototype-only, as with #1047. ## Test plan - [x] All 16 coverage scripts exit 0, "unclassified: none". - [x] Feedback-site tests: 127 passed. - [x] Rendered with Playwright in both themes: - lane 3 Review ("Approve…" under the Discord tally); - lane 4's bar ("Sweep the shelf…" fits on one line); - the phone foot; - the sideways phone. - [ ] **To look at:** in the upright phone drawing, "Sweep the shelf…" wraps to two lines in its button. The app does the same at 390 px, so the drawing is faithful. - [ ] Not opened: about half of the rendered variants. The summary button's change was made after the render. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Prototype: the summary's approve button reads Approve…, as the app's does (D276)
All checks were successful
CI / Bot/backend version sync (pull_request) Successful in 38s
CI / Backend lint (ruff) (pull_request) Successful in 39s
CI / Feedback site tests (pull_request) Successful in 55s
CI / Docker image build (pull_request) Successful in 25s
CI / Does this change touch the recording path? (pull_request) Successful in 27s
CI / Summarisation accuracy eval harness (stub provider) (pull_request) Successful in 1m9s
CI / Synthetic session harness (no GPU, no LLM) (pull_request) Has been skipped
CI / Bot tests and audit (pull_request) Successful in 2m9s
CI / Frontend (shard 2/2) (pull_request) Successful in 10m12s
CI / Frontend (shard 1/2) (pull_request) Successful in 12m31s
CI / End-to-end journeys (desktop) (pull_request) Successful in 13m56s
CI / End-to-end journeys (keyboard) (pull_request) Successful in 15m27s
CI / End-to-end journeys (phone) (pull_request) Successful in 17m26s
CI / Backend migration, tests, and audit (pull_request) Successful in 27m11s
1ec7925fbe
The same button family as the transcript's: ReviewSummary.jsx says
"Approve…" and opens the confirmation that names what approving does.
The confirmation's own title is left as drawn.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
claude-bot deleted branch docs/prototype-wording-d276 2026-10-04 04:14:27 +00:00
Sign in to join this conversation.
No description provided.