Skip to content

fix(review): header fields fit their card at 1280 px – Speichern stays visible - #105

Open
Fluory wants to merge 3 commits into
mainfrom
claude/fix-correction-column-103
Open

Fluory wants to merge 3 commits into
mainfrom
claude/fix-correction-column-103

Conversation

@Fluory

@Fluory Fluory commented Sep 28, 2026 •

Copy link
Copy Markdown
Owner

Warum

Fixes #103 · beim lokalen Start am 2026-09-28 gefunden

Arbeitsstand

  • Ziel: In „Erkannte Angaben“ ist bei 1280 px der „Speichern“-Knopf jeder Korrektur sichtbar; die Tabelle passt in ihre Karte.
  • Nicht-Ziele: Umbau der Korrektur in das Seitenpanel; Layout unter 1280 px.
  • Erledigt: Spalte „Quelle“ aufgelöst („Quelle anzeigen“ unter dem Status, Spaltenkopf „Status und Quelle“), Eingabefeld 10rem statt 11rem, Test-ID am Tabellen-Container, E2E misst den Container, CHANGELOG; frischer Review eingearbeitet (Link-Namen je Feld für Screenreader, Gegenprobe der E2E-Prüfung); CI grün.
  • Offen: Review/Merge durch den Orchestrator.
  • Annahmen: keine.
  • Nächster kleinster Schritt: Review und Merge.

Was ist passiert (Klartext)

Auf der Prüfseite war die Tabelle „Erkannte Angaben“ bei einem üblichen Laptop-Bildschirm etwas zu breit für ihren Kasten. Der Knopf „Speichern“ neben jedem Korrekturfeld lag deshalb außerhalb des sichtbaren Bereichs; man hätte die Tabelle seitlich verschieben müssen. Jetzt steht der Link „Quelle anzeigen“ unter dem Status statt in einer eigenen Spalte, und alles passt – der Knopf ist sichtbar. Für Screenreader sagt jeder dieser Links zusätzlich, zu welchem Feld er gehört.

Plan-Pflicht (SYSTEM.md §4)

  • Kein Auslöser – keine Modulgrenze, öffentliche API, Migration, Auth/Rechte, kein Zahlungs-/Daten-/Infrapfad, höchstens zwei Module, keine Architekturvarianten, umkehrbar
  • Auslöser zutreffend – Impact Manifest ausgefüllt (Plan vor Code)

Geändert

  • src/app/requests/[id]/page.tsx: vier statt fünf Spalten; „Quelle anzeigen“ in der Status-Zelle mit unsichtbarem Zusatz „für ‹Feld›“; data-testid="fields-table-wrap"
  • src/app/review.css: Eingabe 10rem; .source-link statt .source-col
  • tests/e2e/sample-smoke.spec.ts: bei 1280 px scrollWidth - clientWidth <= 0 am Container und rechte Kante von „Speichern“ innerhalb des Containers
  • tests/e2e/review-smoke.spec.ts: klickt „Quelle anzeigen für Ansprechpartner“
  • CHANGELOG.md

Nachweis (SYSTEM.md §11)

  • Ursache gemessen (lokale App, Musterfall „Werk Ost“, 1280 px): vorher Überlauf 98 px, rechte Kante „Speichern“ 863 px bei Container-Rand 785 px. Nachher: 1280 px → Überlauf 0, Knopf 765 ≤ 785; 1440 px → 0, 850 ≤ 870. Bei 1024 px scrollt die Tabelle weiter in ihrem Container (Überlauf 150 px). Das ist erlaubt: Unter 1280 px scrollt der Container, nie die Seite.
  • Gegenprobe (Review): altes Layout von main + nur die Test-ID → sample-smoke rot mit Received: 98; mit diesem PR grün.
  • verify:changed: grün – tsc, eslint sauber; Build ok
  • verify: grün – CI check auf 3df6705 (Lauf 36982814883, Label verify-full)
  • verify:full / E2E-Spec: grün – lokal alle drei Specs (review-smoke, line-items-smoke, sample-smoke) und in CI (Label verify-full)
  • Manueller Prüfnachweis: Screenshot der lokalen App bei 1280 px nach dem Fix: alle Zeilen einzeilig, „Speichern“ sichtbar
  • Frischer Review (P1 vor Ready-for-review; Architektur/API/DB immer): fix(review): header fields fit their card at 1280 px – Speichern stays visible #105 (comment) – nichts blockierend, beide should umgesetzt

Doku-Entscheidung (genau eine)

Entferntes oder Umbenanntes: docs/ + README gegrept, Treffer bereinigt: Spalte „Quelle“ der Prüfseite – in docs/ nicht beschrieben

Dateigrößen und neue Bausteine (SYSTEM.md §7)

Dateien über 500 Zeilen im Diff (Ausnahmen: generierter Code, Lockfiles, Fixtures, Migrationen, Schemas, Ressourcen, Doku, Konfiguration):

  • keine
  • bewusst belassen – Begründung: –
  • im selben PR nach fachlicher Verantwortung geteilt
  • Folge-Issue –

Über 800 Zeilen mit neuer Fachlogik oder über 1000 Zeilen (P1/P2): nicht betroffen

Neue Shared-Komponente, Utility-Datei, Adapter oder fachlicher Service:

  • nein
  • ja – gesucht nach: –

Subagent-Einsätze

  • 1 × frischer Review (read-only, Sonnet) – nichts blockierend, 2 should umgesetzt, nits begründet; siehe PR-Kommentar.

Risiken / offene Punkte


🤖 Generated with Claude Code

https://claude.ai/code/session_01DJ5vaKvTYiMvdngT4d3xo1

…ays visible (#103)

Five columns with a fixed 11rem input were about 820 px wide against 757 px:
the table scrolled inside its container and pushed the save button out of
sight (measured locally: 98 px overflow, button right edge 863 vs 785).
The source link now sits under the status badge (one column less) and the
input is 10rem: no overflow at 1280 and 1440 px. The E2E measures the
table's own scroll container – a click scrolls the button into view and
never noticed.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Fluory Fluory added the verify-full Run verify:full (integration + E2E) in CI label Sep 28, 2026
@vercel

vercel Bot commented Sep 28, 2026

Copy link
Copy Markdown

Deployment failed for project requestflow-ai with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/doc-edit?upgradeToPro=build-rate-limit

… overflowed (#105 review)

- the value cell wraps a long unbroken value (e-mail address) anywhere, and
  the status cell may wrap its hint and link: the table's minimum width no
  longer sits at the card edge (the review measured 3 px reserve)
- the E2E check moves from the stub request (short values, would have passed
  on the old layout) to the sample „Werk Ost“ – the request that overflowed

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@vercel

vercel Bot commented Sep 28, 2026

Copy link
Copy Markdown

Deployment failed for project requestflow with the following error:

Resource is limited - try again in 24 hours (more than 100, code: "api-deployments-free-per-day").

Learn More: https://vercel.com/doc-edit?upgradeToPro=build-rate-limit

…review)

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01DJ5vaKvTYiMvdngT4d3xo1
@vercel

vercel Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
requestflow Ready Ready Preview Oct 2, 2026 8:16am UTC
requestflow-ai Error Error Oct 2, 2026 8:16am UTC

Fluory commented Oct 2, 2026

Copy link
Copy Markdown
Owner Author

Frischer Review (read-only Subagent) – Ergebnis und Umgang

Nichts blockierend. Bestätigt: .source-col hat keine Nutzer mehr; kein Integrationstest und keine andere Spec hängt an den alten Spaltenköpfen; Locators nach Rolle/Test-ID; CHANGELOG im richtigen Abschnitt.

Befund Umgang
should – mehrere gleichlautende Links „Quelle anzeigen“ ohne Feldbezug (Screenreader-Linkliste) behoben in 3df6705: unsichtbarer Zusatz „für ‹Feld›“ (visually-hidden), sichtbarer Text unverändert; review-smoke klickt jetzt „Quelle anzeigen für Ansprechpartner“
should – nicht belegt, dass die neue E2E-Prüfung mit dem alten Layout rot wird belegt: alte page.tsx + review.css von main (nur Test-ID am Container ergänzt), Build, sample-smoke → rot, Received: 98 (Überlauf in px) bei toBeLessThanOrEqual(0); mit diesem PR grün
nit – rechte-Kante-Prüfung von „Speichern“ teils redundant bleibt: prüft das eigentliche Nutzerproblem (Knopf sichtbar) direkt
nit – Zeilen ohne Quelle zeigen kein „–“ mehr bleibt: der Status („fehlt“) sagt es bereits
nit – overflow-wrap: anywhere kann die Wertspalte auf dem Handy schmal machen bleibt: die Tabelle scrollt dort in ihrem Container (Ausnahme unter 1280 px laut PR)
nit – Konflikt mit #97 in der Status-Zelle bekannt, steht unter Risiken

Lokal nach den Änderungen: tsc, eslint sauber, Build ok, alle drei E2E-Specs grün (review-smoke, line-items-smoke, sample-smoke).


Generated by Claude Code

This branch had an error being deployed

1 failed and 1 active deployments
Preview – requestflow — 3df67059 Deployed Oct 2, 2026 by vercel[bot]
Preview – requestflow-ai — 3df67059 Deployed Oct 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

verify-full Run verify:full (integration + E2E) in CI

Projects

None yet

Development

Successfully merging this pull request may close these issues.

fix(review): the correction column of the header fields is cut off at 1280 px

2 participants