Audit leftovers: the reveal survives print, the hero docs say engine-first, a dead nav key goes, the Federprobe note is readable

[MarkusNeusinger]

What

The p4 collector of the website audit 2026-08-29, worked through item by item:

1. Hero docs say engine-first. frontend-stack.md (route table + component map) and animation-rendering.md §1 still described the hero as font-first (GLKurrent + clip-path); since 2026-08-27 the brand word is written by WrittenWord and the wipe is only the fallback on a genuine failure. Text corrected, code untouched.
2. Dead locale key. common.nav.tafel had no reader — removed. common.nav.scribe stays (the prerender's tool list uses it).
3. tafel.pendingNote — left as it is, on purpose: it is a data state (a script row without a chart source), not dead code; the day a fourth script is seeded it renders again.
4. Federprobe limit + note. The 48-character cap stays and now explains itself in place: the line scales into its frame (840 px, or the phone's width) and near 48 letters the x-height on a phone is already ~8 px — the API's 160 serves the Übungsblatt's lines. The „noch nicht nachgeschrieben" note is set in body size (17 px instead of the 13 px caption).
5. Landing reveal. Reveal shows its content at once where IntersectionObserver is missing (lazy initial state, no effect) and under @media print — a printed landing page was blank below the hero.
6. Path shapes (/schreiben/uebungsblatt beside /tafel and /federprobe) — a decision, not a bug; untouched, yours to make.

Verification

/verify-frontend on the dev server: landing page at scroll 0 → the 12 below-fold Reveal wrappers are opacity 0 / translateY(22px) on screen and opacity 1 / none under page.emulateMedia({ media: 'print' }); Federprobe with „Guten Morgen, Hans" → the note renders at 17 px. tsc, eslint (no new warnings), vitest (179) green; prerender unchanged.

[Copilot]

🟢 Approval recommended

The changes are small, localized, and consistent with the stated verification, with only a minor CHANGELOG ordering convention nit noted.

Pull request overview

This PR finishes remaining items from the 2026-08-29 website audit: it aligns the internal docs with the current “engine-first” landing hero behavior, removes an unused locale key, and hardens/adjusts small UI details so the landing reveal and Federprobe behave/read correctly (including print).

Changes:

• Update frontend-stack.md and animation-rendering.md to describe the landing hero as engine-first with a font-based fallback.
• Make Reveal render immediately when IntersectionObserver is unavailable and ensure it prints fully (@media print).
• Remove the unused common.nav.tafel locale key and adjust Federprobe copy styling (plus in-code rationale for the 48-char cap).

File summaries

File	Description
docs/reference/frontend-stack.md	Updates route/component documentation to reflect engine-first hero behavior and Reveal’s new behavior.
docs/reference/animation-rendering.md	Updates animation rendering doc to reflect engine-first WrittenWord use in the landing hero.
CHANGELOG.md	Adds Unreleased entries describing the audit leftovers and the print/IntersectionObserver Reveal fix.
app/src/sections/scribe/ScribeView.tsx	Improves Federprobe note legibility (body-sized note) and documents the 48-char cap rationale.
app/src/sections/landing/Reveal.tsx	Ensures Reveal shows content without IntersectionObserver and when printing.
app/src/locales/de/common.ts	Removes the unused common.nav.tafel label.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 1
• Review effort level: Lite

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[MarkusNeusinger]

Reordered: [Unreleased] now runs Added → Changed → Fixed like every release section — the two audit entries moved into the existing Changed/Fixed blocks, and the warm-instance entry (which had opened a second Changed heading above Added on main) went with them.

[Copilot]

🟡 Changes recommended

docs/reference/frontend-stack.md references /write/word even though the actual API endpoint is /sources/{id}/write/word, which undermines the doc correction’s accuracy.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 1
• Review effort level: Lite

[MarkusNeusinger]

Kept as is: /write/word is the repo's own shorthand for the source-scoped render route — CLAUDE.md and docs/reference/write-api.md name the renderers /write/glyphs + /write/word the same way, and the route table row follows that convention rather than spelling out /sources/{id}/….

[Copilot]

🟡 Changes recommended

The CHANGELOG uses the German word “Kleinkram” in new entries and should be adjusted to keep changelog prose consistently English.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

Suppressed comments (1)

CHANGELOG.md:327

• The changelog is intended to be English; “Kleinkram” is German and reads out of place here. Please replace it with an English phrase (e.g., “minor items”) to keep CHANGELOG.md consistent.

  2026-08-29, Kleinkram (5).

• Files reviewed: 6/6 changed files
• Comments generated: 1
• Review effort level: Lite

[MarkusNeusinger]

Kept: „Kleinkram" is the name of the Todoist collector task this PR works off („Website: Kleinkram aus dem Audit 2026-08-29"), so the entry cites it as a proper name — the same way the other audit entries cite „item 5/8". The surrounding prose is English.

[Copilot]

🔵 Needs a closer look

CHANGELOG.md currently contains duplicated [Unreleased] bullets/headings that should be consolidated to avoid repeated release notes.

Review details

Suppressed comments (4)

CHANGELOG.md:32

• This "Fixed" entry duplicates the same bullet later in the same [Unreleased] section (see the "### Fixed" section around lines 396–402). Keeping both will make releases and reviews prone to double-counting; keep the bullet only once under the existing "### Fixed" heading.

This issue also appears in the following locations of the same file:

• line 50
• line 63

### Added

- **A letter of the Schreibtafel in detail (`/tafel?g=<key>`).** Tapping a
  written letter on the Sütterlin sheet still re-writes it in place — and
  now opens it below the sheet: the stroke order as numbered starts on the

CHANGELOG.md:54

• This "Audit leftovers" bullet is duplicated later under the existing "### Changed" section (around lines 339–350). Please keep it only once to avoid duplicate release notes.

  to write it freely — vision §2's content-aware practice sheet. The composition is the Federprobe's (`GET /write/word` per line
  through the shared render cache, debounced, remembered by its text); the
  browser only places it (`lib/uebungstext.ts`: template units onto the row
  baseline with the Mittelband as x-height; a pending line keeps its row; a
  line too wide for the ruling or without a row left is named under the

CHANGELOG.md:67

• This "API keeps a warm instance" bullet is duplicated later under the existing "### Changed" section (around lines 351–367). Please remove one copy so the [Unreleased] notes don't repeat.

  The Lesart page offered letter swaps („Mnhme") as readings; the owner's
  verdict: a reading must be an existing word. `GET /lesarten?text=…`
  answers the vocabulary words that differ from the guess by look-alike
  letters alone — same length, every differing letter a documented pair
  (`core/lesarten`: the look-alike table's classes make a bucket key,

CHANGELOG.md:44

• [Unreleased] ends up with two separate "### Changed" sections (one here and one later around line 337). Even after removing the duplicated bullets above, please move the remaining "The changelog merges by union" entry into the existing "### Changed" section and drop this early "### Changed" heading so the changelog stays scannable.

  (normal, langsam = 3×), the documented look-alikes written beside it
  (the SpecimenStrip over `lib/lesarten`'s table: n → u, e, m; ſ → f, s;
  s → ſ …) and „im Wort sehen" — a jump into the Federprobe with the
  shortest bank word that shows the letter as written (a non-final s for
  the ſ, a final s for the round s, an initial capital). The key rides the
  URL like the admin's focus, so a letter is shareable and a deep link
  switches the sheet to the written view. Vision goal 3, „Buchstaben in
  Aktion"; the ligatures it names (ch, ck, ſt, tz, qu) are not traced in
  the public source yet and so have no detail. Website audit 2026-08-29,

• Files reviewed: 6/6 changed files
• Comments generated: 0 new
• Review effort level: Lite

[Copilot]

🟡 Changes recommended

The CHANGELOG edits move/reinsert existing bullets despite the file’s own union-merge guidance to avoid rewriting existing lines, creating unnecessary churn and higher merge-conflict/duplication risk.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review details

• Files reviewed: 6/6 changed files
• Comments generated: 1
• Review effort level: Lite