Rapport: kritiek-pad-vinkje verborgen bij het resourcediagram (#113, nabespreking)

[Nozzit]

What and why

Refs #113 — opmerking 4 van manuvarkey na de merge van #132: het vinkje Kritiek pad heeft bij het resourcediagram geen betekenis. Dat klopt, en mijn eerdere tegenwerping was fout: die leunde op een verouderd commentaar in printPreview.ts dat beweerde dat de balklaag bij een uitgezet vinkje neutraal tekent. Gecontroleerd in de code: de balken lopen door computeBarColors (barColors.ts, criticalFill in de modus Kritiek pad, een rode rand daarbuiten) en kijken niet naar het vinkje; showCritical stuurt alleen de kleur van de relatielijnen en de legendaregel. Bij het resourcediagram, dat geen relatielijnen tekent, nam het vinkje dus uitsluitend de legendaregel weg terwijl de balken rood bleven — misleidend.

• reportTypeShowsCriticalToggle (reportSettings.ts): hetzelfde predicaat als reportTypeDrawsRelations. ReportPanel verbergt het vinkje voor het resourcediagram en forceert showCritical op true, zodat de legenda de rode balken blijft verklaren; het gewone Gantt-rapport verandert niet.
• Het verouderde renderercommentaar is rechtgezet.
• Gidsen nl+en: de zin bij de relatielijnen zei dat kritiek pad uit "ook" de lijnen neutraal maakt (de balken doen dat sinds de balkkleurkeuze niet meer) en de resourcediagram-sectie noemt nu drie uitzonderingen. De twaalf vertaalde gidsen volgen de maandelijkse vertaalronde (verify:docs eist alleen kop-/linkpariteit).
• CLAUDE.md: één zin in de rapportparagraaf.

Manu's punten 1–3 (units/dag en curve per rij, groepering per resourcetype, rapportageperiode zoals #134) zitten hier bewust niet in; dat zijn losse vervolgstappen.

How it was verified

• npm run verify green — lokaal: typecheck, lint, verify:docs (34 × 14), de volledige planningssuite (560/560 + tijdzone-matrix; één timingcheck, task-grid-performance, gaf 557 ms > 500 ms terwijl de browsersuite er tegelijk naast draaide en haalt in isolatie 274 ms — code die dit diff niet raakt), en de browserspecs resource-gantt-report.spec.ts + report-footer.spec.ts (3/3).
• Regressie: tests/planning/check-report-resource-gantt-setting.ts bewaakt het predicaat; tests/browser/resource-gantt-report.spec.ts bewaakt dat het vinkje bij het Gantt-rapport wél en bij het resourcediagram níét in het paneel staat.

Does this touch

• Project data — n.v.t.
• Scheduling logic — n.v.t.
• User-visible text — geen nieuwe sleutels; alleen een bestaand vinkje dat bij één rapporttype verborgen wordt.
• @tauri-apps/* — n.v.t.

Documentation

• public/docs/{nl,en}/gids-rapporten-printen.md (relatielijnen-alinea, resourcediagram-sectie), CLAUDE.md.
• docs/CHANGELOG.md bewust niet (alleen bij een release).

🤖 Generated with Claude Code

https://claude.ai/code/session_014zosxbSXvi3YSGmeN1MsQq

---

Generated by Claude Code