fix(finding): inhoud-bewuste auto-fit voor de bevinding-header (#1282) #1288

Merged

brenno merged 1 commit from fix/1282-finding-header-autofit into main

2026-08-05 23:12:25 +00:00

brenno commented

2026-08-05 22:28:28 +00:00

Owner

Een finding-header (ocideck_finding_role: header) rendeerde veel te groot: alle fontgroottes waren vast × de #1163-constante 0.84, zonder inhoud-bewuste krimpstap. Een dichte header liep over en werd door BoxFit.scaleDown verkleind én linksboven geparkeerd (de dia maar deels benut); ~50% was via de pagineringsvloer zelfs onbereikbaar.

Nieuwe top-level findingHeaderFitScale in slide_layout_metrics.dart meet de gepagineerde pagina-inhoud (kaart + de secties van díe pagina) tegen de diahoogte en geeft een vermenigvuldiger in (0, maxScale]: een dichte header krimpt tot hij past, een schaarse wordt op het plafond gehouden (nooit vergroot boven de #1163-look).
_FindingPreview past die schaal toe (via fitScaleOverride); de gedeelde constante kFindingBaseFontScale voorkomt drift met #1163.
Paginering blijft intact: er wordt op de gepagineerde inhoud gemeten, dus een finding die legitiem over meerdere dia's gaat blijft splitsen. Buiten scope gehouden: het body-kostenmodel (#1198).

Regressietest tegen een echte render (finding_header_autofit_test.dart): een dichte F-01-header past nu binnen de dia (geen scaffold-krimp meer) én rendert onder een bovengrens — eerst-rood geverifieerd (zonder de fit valt scaleDown naar 0.69 en faalt de assertie). Bestaande paginering-/breedte-tests (#1147/#1198) blijven groen.

Closes #1282

🤖 Generated with Claude Code

Een `finding`-header (`ocideck_finding_role: header`) rendeerde veel te groot: alle fontgroottes waren vast × de #1163-constante 0.84, zonder inhoud-bewuste krimpstap. Een dichte header liep over en werd door `BoxFit.scaleDown` verkleind én linksboven geparkeerd (de dia maar deels benut); ~50% was via de pagineringsvloer zelfs onbereikbaar. - Nieuwe top-level `findingHeaderFitScale` in `slide_layout_metrics.dart` meet de **gepagineerde** pagina-inhoud (kaart + de secties van díe pagina) tegen de diahoogte en geeft een vermenigvuldiger in `(0, maxScale]`: een dichte header krimpt tot hij past, een schaarse wordt op het plafond gehouden (nooit vergroot boven de #1163-look). - `_FindingPreview` past die schaal toe (via `fitScaleOverride`); de gedeelde constante `kFindingBaseFontScale` voorkomt drift met #1163. - **Paginering blijft intact**: er wordt op de gepagineerde inhoud gemeten, dus een finding die legitiem over meerdere dia's gaat blijft splitsen. Buiten scope gehouden: het body-kostenmodel (#1198). Regressietest tegen een **echte render** (`finding_header_autofit_test.dart`): een dichte F-01-header past nu binnen de dia (geen scaffold-krimp meer) én rendert onder een bovengrens — eerst-rood geverifieerd (zonder de fit valt `scaleDown` naar 0.69 en faalt de assertie). Bestaande paginering-/breedte-tests (#1147/#1198) blijven groen. Closes #1282 🤖 Generated with [Claude Code](https://claude.com/claude-code)

brenno added 1 commit

2026-08-05 22:28:28 +00:00

fix(finding): inhoud-bewuste auto-fit voor de bevinding-header zodat een dichte header niet te groot rendert (#1282 )

static-gate / static-gate (pull_request) Failing after 1m21s

Details

scans / scans (pull_request) Successful in 1m35s

Details

ca53322fc8

Meet de gepagineerde header-inhoud en schaalt de vaste 0.84-fontgrootte omlaag
tot hij past, met een bovengrens zodat een schaarse header niet opblaast.
Paginering blijft intact; de header-kostentest meet met fitScaleOverride: 1.0.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno force-pushed fix/1282-finding-header-autofit from ca53322fc8

static-gate / static-gate (pull_request) Failing after 1m21s

Details

scans / scans (pull_request) Successful in 1m35s

Details

to 0655b0e34f

scans / scans (pull_request) Successful in 1m37s

Details

static-gate / static-gate (pull_request) Failing after 4m4s

Details

2026-08-05 22:40:18 +00:00

Compare