docs(M42): spec en plan voor de Vis-design-migratie #292

Merged

janpeter merged 10 commits from claude/design-systems-knowledge-ai-8eb6f4 into main 2026-10-04 09:53:38 +02:00

Conversation 2 Commits 10 Files changed 3 +1092

janpeter commented 2026-10-03 16:13:00 +02:00

Owner

Copy link

Alleen documentatie: de spec en het implementatieplan voor M42, het omzetten van de UI van Scrum4Me en scrum4me-workers naar het Vis-design (web, iPhone, Watch).

Inhoud

• docs/specs/2026-10-03-vis-design-migratie-design.md — spec, revisie 5. Dubbel GO (Codex en Claude) na vier reviewrondes; het reviewverslag staat onderaan.
• docs/plans/M42-vis-design-migratie.md — implementatieplan, revisie 3. Dubbel GO na twee reviewrondes; het reviewverslag staat onderaan.
• docs/INDEX.md — opnieuw gegenereerd.

Waarom nu

De taken van sprint S-2026-10-03-m42-vis (PBI-177, ST-1623 t/m ST-1628) verwijzen naar deze twee documenten. Met deze merge staan ze op main, zodat de werkbranches ze meekrijgen.

Niet in deze PR

Geen codewijziging. De uitvoering van M42 is nog niet gestart en wacht op een aparte opdracht.

Verificatie

npm run docs: index gegenereerd en alle links geldig (307 bestanden).

🤖 Generated with Claude Code

Alleen documentatie: de spec en het implementatieplan voor M42, het omzetten van de UI van Scrum4Me en scrum4me-workers naar het Vis-design (web, iPhone, Watch). ## Inhoud - `docs/specs/2026-10-03-vis-design-migratie-design.md` — spec, revisie 5. Dubbel GO (Codex en Claude) na vier reviewrondes; het reviewverslag staat onderaan. - `docs/plans/M42-vis-design-migratie.md` — implementatieplan, revisie 3. Dubbel GO na twee reviewrondes; het reviewverslag staat onderaan. - `docs/INDEX.md` — opnieuw gegenereerd. ## Waarom nu De taken van sprint `S-2026-10-03-m42-vis` (PBI-177, ST-1623 t/m ST-1628) verwijzen naar deze twee documenten. Met deze merge staan ze op `main`, zodat de werkbranches ze meekrijgen. ## Niet in deze PR Geen codewijziging. De uitvoering van M42 is nog niet gestart en wacht op een aparte opdracht. ## Verificatie `npm run docs`: index gegenereerd en alle links geldig (307 bestanden). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

janpeter added 9 commits 2026-10-03 16:13:00 +02:00

docs(M42): spec en plan voor de Vis-design-migratie (ronde 1 ter review) ... 0fa2334639

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): spec revisie 2 na reviewronde 1 ... b257a9b322

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): spec revisie 3 na reviewronde 2 (B6 contrasteis) ... ab205657dd

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): spec revisie 4 na reviewronde 3 ... 700ec43be8

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): spec dubbel GO in ronde 4; MINORs verwerkt (revisie 5) ... 0c332972e4

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): implementatieplan op basis van de spec (ronde 1 ter review) ... 5354215668

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): plan revisie 2 na reviewronde 1 ... a39c71be6b

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): plan dubbel GO in ronde 2; MINORs verwerkt (revisie 3) ... 67020069a4

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

docs(M42): telling tekstlinks gelijk aan de controle (42) ... 561159c96c

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

s4m-codex-reviewer reviewed 2026-10-03 16:17:09 +02:00

s4m-codex-reviewer left a comment

Copy link

COMMENT

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

De diff bevat uitsluitend documentatie. De tokenarchitectuur en bestaande componentpatronen blijven behouden; de overstap krijgt een ADR en bijgewerkte stylingdocumentatie. Geen blokkerende findings.

Findings

• MINOR — docs/specs/2026-10-03-vis-design-migratie-design.md:203: De bindende spec noemt 44 tekstlinkplekken en 10 hoverplekken, terwijl taak 1.3 van het plan 42 regels, 6 te wijzigen hoverregels en expliciete uitzonderingen beschrijft. Synchroniseer de spec met deze preciezere inventaris zodat de acceptatiescope eenduidig is.
• MINOR — docs/plans/M42-vis-design-migratie.md:441: Increment 5 wacht alleen op PR A en C, maar taak 5.3 sluit de sprint af. PR B (workers) kan dan nog openstaan. Neem afronding van PR B expliciet op als voorwaarde voor het afsluiten van de gehele migratie.

Verificatie

PR-head bevestigd op 561159c96ceeb6775914f6dae1943a7bedf71732. Statuskleuren op eigen tinten nagerekend: minimum 4,553:1 licht en 4,523:1 donker; invoerranden minimaal 3,024:1 en 3,158:1. De geplande contrasttest, catalogustest en screenshotcontroles passen bij de migratie. Geen build of runtime-tests uitgevoerd: deze PR wijzigt geen uitvoerbare code en implementeert de migratie nog niet.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. De diff bevat uitsluitend documentatie. De tokenarchitectuur en bestaande componentpatronen blijven behouden; de overstap krijgt een ADR en bijgewerkte stylingdocumentatie. Geen blokkerende findings. ## Findings - **MINOR** — `docs/specs/2026-10-03-vis-design-migratie-design.md:203`: De bindende spec noemt 44 tekstlinkplekken en 10 hoverplekken, terwijl taak 1.3 van het plan 42 regels, 6 te wijzigen hoverregels en expliciete uitzonderingen beschrijft. Synchroniseer de spec met deze preciezere inventaris zodat de acceptatiescope eenduidig is. - **MINOR** — `docs/plans/M42-vis-design-migratie.md:441`: Increment 5 wacht alleen op PR A en C, maar taak 5.3 sluit de sprint af. PR B (workers) kan dan nog openstaan. Neem afronding van PR B expliciet op als voorwaarde voor het afsluiten van de gehele migratie. ## Verificatie PR-head bevestigd op `561159c96ceeb6775914f6dae1943a7bedf71732`. Statuskleuren op eigen tinten nagerekend: minimum 4,553:1 licht en 4,523:1 donker; invoerranden minimaal 3,024:1 en 3,158:1. De geplande contrasttest, catalogustest en screenshotcontroles passen bij de migratie. Geen build of runtime-tests uitgevoerd: deze PR wijzigt geen uitvoerbare code en implementeert de migratie nog niet.

janpeter added 1 commit 2026-10-03 16:20:13 +02:00

docs(M42): verwerk planbevindingen en actuele sprintbinding 75d74388d1

s4m-codex-reviewer reviewed 2026-10-03 16:28:27 +02:00

s4m-codex-reviewer left a comment

Copy link

COMMENT

geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Deze PR voegt uitsluitend documentatie toe: de migratiespec, het implementatieplan en indexverwijzingen. Geen blokkerende bevindingen; het plan behoudt het bestaande tokensysteem en beschrijft tests, platformverificatie en visuele acceptatie. De gedimde-tekstafwijking van B6 is expliciet beschreven en afgedekt door de poort van taak 0.4.

Findings:

• MINOR — docs/plans/M42-vis-design-migratie.md:1; docs/specs/2026-10-03-vis-design-migratie-design.md:1: YAML-frontmatter ontbreekt. De index toont daardoor geen status/datum; scripts/sync-product-docs.ts valt terug op de bestandsnaam als titel en draft als status. Voeg title, status en last_updated toe voor consistente documentmetadata.

Verificatie: PR-head bevestigd als 75d74388d1f190268f4bdfea75843d55bfc7adec; beide nieuwe documenten komen exact overeen met de aangeleverde diff. npm run docs geslaagd; relatieve links van beide documenten afzonderlijk gecontroleerd. Onafhankelijke WCAG-berekening: status/tint minimaal 4,553:1 (licht), 4,523:1 (donker); control-border minimaal 3,024:1 en 3,158:1. Geen applicatietests of build uitgevoerd voor deze documentatie-only diff; de beschreven toekomstige tests zijn nog geen implementatiebewijs.

# COMMENT geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. Deze PR voegt uitsluitend documentatie toe: de migratiespec, het implementatieplan en indexverwijzingen. Geen blokkerende bevindingen; het plan behoudt het bestaande tokensysteem en beschrijft tests, platformverificatie en visuele acceptatie. De gedimde-tekstafwijking van B6 is expliciet beschreven en afgedekt door de poort van taak 0.4. Findings: - **MINOR — docs/plans/M42-vis-design-migratie.md:1; docs/specs/2026-10-03-vis-design-migratie-design.md:1:** YAML-frontmatter ontbreekt. De index toont daardoor geen status/datum; `scripts/sync-product-docs.ts` valt terug op de bestandsnaam als titel en `draft` als status. Voeg `title`, `status` en `last_updated` toe voor consistente documentmetadata. Verificatie: PR-head bevestigd als `75d74388d1f190268f4bdfea75843d55bfc7adec`; beide nieuwe documenten komen exact overeen met de aangeleverde diff. `npm run docs` geslaagd; relatieve links van beide documenten afzonderlijk gecontroleerd. Onafhankelijke WCAG-berekening: status/tint minimaal 4,553:1 (licht), 4,523:1 (donker); control-border minimaal 3,024:1 en 3,158:1. Geen applicatietests of build uitgevoerd voor deze documentatie-only diff; de beschreven toekomstige tests zijn nog geen implementatiebewijs.

janpeter referenced this pull request 2026-10-04 08:23:09 +02:00

M42 — Vis-thema voor Scrum4Me-web (PR A) #293

janpeter referenced this pull request 2026-10-04 08:23:51 +02:00

M42 — Vis-thema voor iPhone en Watch (PR C) #294

janpeter merged commit 3e52dba44d into main 2026-10-04 09:53:38 +02:00

janpeter referenced this pull request from a commit 2026-10-04 09:53:40 +02:00

Merge pull request 'docs(M42): spec en plan voor de Vis-design-migratie' (#292) from claude/design-systems-knowledge-ai-8eb6f4 into main

janpeter referenced this pull request 2026-10-04 10:44:46 +02:00

M42 — Vis-thema voor Scrum4Me-web (PR A) #293

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

severity/s3

  

severity/s4

No labels

severity/s3

severity/s4

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

janpeter/Scrum4Me!292

No description provided.