M42 — Vis-thema voor Scrum4Me-web (PR A) #293
No reviewers
Labels
No labels
severity/s3
severity/s4
No milestone
No project
No assignees
2 participants
Notifications
Due date
No due date set.
Dependencies
No dependencies set.
Reference
janpeter/Scrum4Me!293
Loading…
Add table
Add a link
Reference in a new issue
No description provided.
Delete branch "feat/m42-vis-web"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
De Scrum4Me-webinterface gebruikt het Vis-thema in licht en donker: kleuren, Public Sans, Source Serif 4 en Geist Mono, pilknoppen, één ember-hoofdactie volgens de JP-keuzes, herkenbare links, invoerveldranden, statuslabels en de eigen toaster. Stylingdocumentatie en ADR zijn bijgewerkt.
De volledige T-1983-routecontrole is afgerond: alle 41 routepatronen in beide modi, inclusief admin, mobiel, meldingen en de actieve dialogen. Er zijn 151 originele PNGs met hashes en een geschoonde routematrix in
docs/plans/M42-bewijs/webroutecontrole.mdenincrement-1/. De controle herstelde aanvullende tekstlinks, de sprintdatum-submit, de onbedoelde mobiele +PBI-ember en donkere native kalendericonen. JP autoriseerde daarnaast één mobiele storykolom onder 1024 px; desktop houdt drie. Hiermee is de bestaande onzichtbaarheid van mobiele storytitels (ISS-5) hersteld. ISS-4, de bestaande teller ‘9 van 8 folders’, blijft afzonderlijk open.Validatie: lint zonder fouten (6 bestaande waarschuwingen), typecheck, volledige Vitest-suite met 3.884 geslaagde en 29 overgeslagen tests (maxWorkers=2), productiebuild en documentlinks in 308 bestanden. Browserreadback: native date color-scheme dark met zichtbare kalendericonen; mobiele titelbreedte 231 px en drie desktopkolommen; juiste fontfamilies en underline-offset 4 met ember-hover. Onafhankelijke review van de aanvullende complete bronpatch: GO, 0 Critical / 0 Important / 0 Minor. Sourcepatch-SHA256: fb9b492f2aa86e8b3e2eb39d1dab4f1d7f3e45bfe03ac8db7ce8726dd5cb8ee3.
De controle gebruikt lokale proefaccounts en fixtures; er zijn geen agentjobs gestart. De tijdelijke Hub-proefserver gebruikte uitsluitend loopback, zonder APNs of Watch-direct; de oorspronkelijke omgeving is hersteld. De dark-klasse is uitsluitend in de testbrowser gebruikt. CI, merge en deployment zijn afzonderlijke stappen. Workers PR B wacht op merge van deze PR A.
Plan: #292. iPhone/Watch: #294. PBI-177, ST-1623 en ST-1624.
Reviewafhandeling (2026-10-04): actuele main
3e52dba44dce4cad4b15500a080a29a531eeaefais via merge geïntegreerd in head32125e87d93e1608a80f334ddcf0836174d14154. De twee documentconflicten zijn opgelost: upstreamplan plus de bestaande JP-besluitregel blijven behouden; de index is opnieuw gegenereerd. Lint, typecheck, volledige suite (3.884 geslaagd /29 overgeslagen), productiebuild en documentlinks zijn opnieuw groen. Alle151 screenshot-hashes zijn ongewijzigd.Gepinde bronnen: M42-plan, Vis-spec, reviewafhandeling. De volledige lokale verificatie vervangt geen nieuw onafhankelijk reviewer-verdict.
WIP: M42 — Vis-thema voor Scrum4Me-web (PR A)to M42 — Vis-thema voor Scrum4Me-web (PR A)REQUEST_CHANGES
geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Findings
docs/INDEX.md:92,docs/plans/M42-vis-design-migratie.md:275: de PR conflicteert met de actuelemain(3e52dba4), waarin PR #292 inmiddels is gemerged. Forgejo meldtmergeable=false; een onafhankelijke driewegmerge vanaf630f6eaebevestigt conflicten in de documentindex en een add/add-conflict in het M42-plan. Integreermain, behoud het aanvullende JP-besluit over de mobiele storykolom en genereer de index opnieuw. De PR kan in deze toestand niet worden gemerged.Beoordeling en verificatie
Beoordeeld op head
964ec537a6e3b10902222bf5794855cbcd0989f6. De themamigratie behoudt de bestaande token- en Base UI-architectuur; kleurwaarden, fonts, veldranden en actievarianten sluiten aan op de nieuwe ADR en stylingdocumentatie in de PR. De mobiele layoutuitzondering is expliciet gedocumenteerd. Geen aanvullende blokkerende broncodebevinding gevonden.Onafhankelijk uitgevoerd: lint (0 fouten, 6 waarschuwingen), typecheck na generatie van Prisma-client en handleiding (exit 0), 760 contrastberekeningen (alle geslaagd), 151 screenshot-hashes (alle correct), documentlinks (308 bestanden, geen fouten). Screenshots van desktopbacklog en mobiele stories visueel bekeken.
Vitest kon hier niet starten doordat
/tmpmetnoexecis gemount en de native Rolldown-binding niet geladen kan worden. De volledige testsuite en productiebuild zijn daarom niet onafhankelijk bevestigd; het aangeleverde verslag meldt 3884 geslaagde tests, 29 overgeslagen tests en een geslaagde build. Dit is een verificatiebeperking van de reviewomgeving, geen aangetoonde codefout.Verdict: COMMENT
geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.
Beoordeeld tegen commit
32125e87d93e1608a80f334ddcf0836174d14154, de productdocumentatie en de nieuwe Vis-ADR/stylingregels. Geen blokkerende bevindingen gevonden; wel een kleine inconsistentie.underline underline-offset-4enhover:decoration-ember; de geselecteerde revisie mist ook de hover-onderstreping. Hetzelfde geldt voor de commitlink incomponents/shared/story-log.tsx:92. Nu primary gelijk is aan foreground en secondary secundaire inkt is, blijft de herkenbaarheid van deze tekstlinks achter bij de vaste regel uit ADR-0016. Trek de linkklassen hier gelijk.Verificatie: exacte PR-head opgehaald; 761 contrast-/rekencontroles afzonderlijk uitgevoerd en geslaagd; ESLint op app/componenten en gewijzigde tests: 0 fouten, 3 bestaande waarschuwingen; 308 documentbestanden zonder linkfouten; 151 screenshot-hashes correct. Mobiele storyweergave en documentdetail steekproefsgewijs visueel bekeken.
Vitest kon in deze reviewomgeving niet starten:
/tmpisnoexec(Permission denied / native binding failed to map). Daardoor zijn componenttests en de volledige verify/build hier niet onafhankelijk bevestigd. De vastgelegde lokale suite/build-resultaten zijn gelezen, maar gelden niet als een eigen succesvolle run. Daarom geen APPROVED.