fix(finding): header-dia gebruikt halve breedte in editor-preview, wél goed in presentatie

brenno commented

2026-08-03 08:37:57 +00:00

Owner

Wat is er mis

Bij het bouwen van een presentatie (de dia-preview links in de editor) rendert de finding-header-dia maar op ongeveer de halve dia-breedte: de severity-headerkaart en de meta staan samengedrukt in de linkerhelft in plaats van de volledige kolom te vullen. In de presentatiemodus rendert dezelfde dia wél correct, over de volle breedte.

Reproductie

Open Proefpresentatie.md (de proefpresentatie met de MIAUW-bevindingen).
Ga naar dia 29 — F-01 · Slim kattenluik met ingebakken sleutel (_class: finding logo-safe, ocideck_finding_role: header).
Kijk naar de dia-preview in het editor-gedeelte (links): de headerkaart benut maar de halve breedte.
Start de presentatie op dezelfde dia: daar klopt de breedte wel.

Waarom dit bekend voorkomt

Dit is een regressie van een eerder opgeloste klasse. De finding-headerkaart is al eens gecorrigeerd omdat hij op de verkeerde breedte rekende:

7bfe39b2 fix(finding): header card gebruikt slidebreedte in plaats van volledige w
a77bd9d6 fix(finding): pagineer lange bevindingen op volle breedte (#950)

Betrokken bestand: lib/widgets/slides/previews/finding_preview.dart (_headerCard, plus de breedte/constraint waarmee de editor de preview inbedt). Het verschil editor-vs-presentatie wijst op een breedte- of constraint-mismatch tussen het editor-preview-pad en het presentatiepad, niet op de kaart zelf.

Bug ⇒ regressietest

Dit hoort een regressietest te krijgen die de eerdere fix bewaakt: render de finding-header op het editor-preview-pad en toets dat de kaart de volledige kolombreedte benut (niet ~0,5·w). Zie de bestaande test(finding)-regressies (o.a. #950) voor de vorm.

## Wat is er mis Bij het bouwen van een presentatie (de dia-preview links in de editor) rendert de **finding-header-dia** maar op ongeveer de halve dia-breedte: de severity-headerkaart en de meta staan samengedrukt in de linkerhelft in plaats van de volledige kolom te vullen. In de **presentatiemodus** rendert dezelfde dia wél correct, over de volle breedte. ## Reproductie 1. Open `Proefpresentatie.md` (de proefpresentatie met de MIAUW-bevindingen). 2. Ga naar dia 29 — `F-01 · Slim kattenluik met ingebakken sleutel` (`_class: finding logo-safe`, `ocideck_finding_role: header`). 3. Kijk naar de dia-preview in het editor-gedeelte (links): de headerkaart benut maar de halve breedte. 4. Start de presentatie op dezelfde dia: daar klopt de breedte wel. ## Waarom dit bekend voorkomt Dit is een **regressie** van een eerder opgeloste klasse. De finding-headerkaart is al eens gecorrigeerd omdat hij op de verkeerde breedte rekende: - `7bfe39b2 fix(finding): header card gebruikt slidebreedte in plaats van volledige w` - `a77bd9d6 fix(finding): pagineer lange bevindingen op volle breedte` (#950) Betrokken bestand: `lib/widgets/slides/previews/finding_preview.dart` (`_headerCard`, plus de breedte/constraint waarmee de editor de preview inbedt). Het verschil editor-vs-presentatie wijst op een breedte- of constraint-mismatch tussen het editor-preview-pad en het presentatiepad, niet op de kaart zelf. ## Bug ⇒ regressietest Dit hoort een regressietest te krijgen die de eerdere fix bewaakt: render de finding-header op het editor-preview-pad en toets dat de kaart de volledige kolombreedte benut (niet ~0,5·w). Zie de bestaande `test(finding)`-regressies (o.a. #950) voor de vorm.

brenno added the

bug

in-progress

labels

2026-08-03 08:37:57 +00:00

brenno commented

2026-08-03 08:51:13 +00:00

Author

Owner

Opgepakt. Tak: fix/finding-header-editor-width-1147. Verwachte reikwijdte: lib/widgets/slides/previews/finding_preview.dart (breedte-/constraint-pad van de header) + regressietest in test/finding_preview_test.dart.

brenno referenced this issue from a commit

2026-08-03 10:24:20 +00:00

fix(finding): toon eerste pagina in enkele-dia-preview i.p.v. hele bevinding (#1147)

brenno referenced this issue

2026-08-03 10:25:34 +00:00

fix(finding): toon eerste pagina in enkele-dia-preview i.p.v. hele bevinding (#1147) #1152

brenno referenced this issue from a commit

2026-08-03 10:30:20 +00:00

Merge pull request 'fix(finding): toon eerste pagina in enkele-dia-preview i.p.v. hele bevinding (#1147)' (#1152) from fix/finding-header-editor-width-1147 into main

brenno commented

2026-08-03 10:31:14 +00:00

Author

Owner

Opgelost op main in c6820b8e (merge 567c9d40, PR #1152).

Oorzaak bleek niet de headerkaart zelf maar het renderpad: presenteren, exporteren en de gepagineerde hoofdpreview draaien expandFindingsForRender (pagineert een te lange bevinding over meerdere dia's op volle breedte), maar de enkele-dia-previews — de dia-strook links en de in-editor live-preview — renderen één brondia en liepen door die blinde vlek heen. Een te lange bevinding werd daar door de FittedBox uniform omlaaggeschaald en zo ook smaller (kaart + meta in de linkerhelft).

Fix: firstRenderPageSpec toont in die previews nu de eerste pagina, gelijk aan presenter/export. De tekst wordt niet groter; alleen de juiste pagina wordt getoond.

Regressietests: firstRenderPageSpec (fits/overflow) in finding_pagination_test, en de kaartbreedte op het enkele-dia-pad in finding_preview_test.

Niet in deze fix: de pagineer-calibratie zelf (een volle pagina mag nog steeds tot ~0,70·breedte schalen — ongewijzigd gedrag van #950).

Opgelost op main in c6820b8e (merge 567c9d40, PR #1152). Oorzaak bleek niet de headerkaart zelf maar het renderpad: presenteren, exporteren en de gepagineerde hoofdpreview draaien `expandFindingsForRender` (pagineert een te lange bevinding over meerdere dia's op volle breedte), maar de enkele-dia-previews — de dia-strook links en de in-editor live-preview — renderen één brondia en liepen door die blinde vlek heen. Een te lange bevinding werd daar door de FittedBox uniform omlaaggeschaald en zo ook smaller (kaart + meta in de linkerhelft). Fix: `firstRenderPageSpec` toont in die previews nu de eerste pagina, gelijk aan presenter/export. De tekst wordt niet groter; alleen de juiste pagina wordt getoond. Regressietests: `firstRenderPageSpec` (fits/overflow) in finding_pagination_test, en de kaartbreedte op het enkele-dia-pad in finding_preview_test. Niet in deze fix: de pagineer-calibratie zelf (een volle pagina mag nog steeds tot ~0,70·breedte schalen — ongewijzigd gedrag van #950).

brenno

2026-08-03 10:31:16 +00:00