fix(split): smallere per-pagina-drempel voor de bulletsImage-split (#1279) #1291

Closed
brenno wants to merge 1 commit from fix/1279-split-autofit into main
Owner

Een overvolle split-slide (bullets + afbeelding) die je over meerdere slides verdeelde, liet het lettertype niet meegroeien: de deel-slides bleven klein met onbenutte ruimte. Oorzaak: alle pagina's van een split-run delen bewust één grootte — het minimum over de run, de vólste pagina (sharedSplitFitScale, een getest invariant). Bij acht lange bullets in de door de afbeelding versmalde tekstkolom is die vólste pagina zo vol dat de gedeelde schaal ver onder de referentiegrootte zakt.

Fix (ontwerp-behoudend): een aparte, kleinere per-pagina-drempel voor de bulletsImage-split — kSplitImageColumnBulletTarget = 5 i.p.v. de volle-breedte kSingleColumnBulletWarningCount (8) — alleen voor slide.type.splitWithImage. Minder bullets per pagina maakt de vólste pagina minder vol, tilt de gedeelde schaal omhoog en vult de ruimte, zonder het uniforme-schaal-invariant los te laten (elke pagina blijft even groot; shared_split_fit_scale_test blijft groen).

Empirisch (59 alinea-lange bullets, --image-width: 40%): bij 8/pagina bleef de gedeelde schaal op ~0.71; bij 5/pagina stijgt hij naar ~0.79 terwijl het aantal pagina's maar van 8 naar 12 groeit. Vijf is ~60% van acht, in de pas met de halve kolombreedte.

Regressietest tegen een echte render (split_image_autofit_test.dart): splitst een overvolle beeld-split, rendert een vervolgpagina, en assert dat de effectieve fontgrootte boven een vloer uitkomt — eerst-rood geverifieerd (met de oude drempel 8 zakt hij eronder en faalt de assert).

Closes #1279

🤖 Generated with Claude Code

Een overvolle `split`-slide (bullets + afbeelding) die je over meerdere slides verdeelde, liet het lettertype niet meegroeien: de deel-slides bleven klein met onbenutte ruimte. Oorzaak: alle pagina's van een split-run delen bewust één grootte — het minimum over de run, de vólste pagina (`sharedSplitFitScale`, een getest invariant). Bij acht lange bullets in de door de afbeelding versmalde tekstkolom is die vólste pagina zo vol dat de gedeelde schaal ver onder de referentiegrootte zakt. **Fix (ontwerp-behoudend):** een aparte, kleinere per-pagina-drempel voor de bulletsImage-split — `kSplitImageColumnBulletTarget = 5` i.p.v. de volle-breedte `kSingleColumnBulletWarningCount` (8) — alleen voor `slide.type.splitWithImage`. Minder bullets per pagina maakt de vólste pagina minder vol, tilt de gedeelde schaal omhoog en vult de ruimte, **zonder het uniforme-schaal-invariant los te laten** (elke pagina blijft even groot; `shared_split_fit_scale_test` blijft groen). Empirisch (59 alinea-lange bullets, `--image-width: 40%`): bij 8/pagina bleef de gedeelde schaal op ~0.71; bij 5/pagina stijgt hij naar ~0.79 terwijl het aantal pagina's maar van 8 naar 12 groeit. Vijf is ~60% van acht, in de pas met de halve kolombreedte. Regressietest tegen een **echte render** (`split_image_autofit_test.dart`): splitst een overvolle beeld-split, rendert een vervolgpagina, en assert dat de effectieve fontgrootte boven een vloer uitkomt — eerst-rood geverifieerd (met de oude drempel 8 zakt hij eronder en faalt de assert). Closes #1279 🤖 Generated with [Claude Code](https://claude.com/claude-code)
fix(split): smallere per-pagina-bulletdrempel voor de bulletsImage-split zodat de deel-slides de ruimte benutten (#1279)
All checks were successful
scans / scans (pull_request) Successful in 2m40s
static-gate / static-gate (pull_request) Successful in 5m2s
5fa21a710e
De afbeelding knijpt de tekstkolom, dus vult de volle-breedte-drempel (8) de
smalle kolom over en houdt de gedeelde split-schaal onnodig klein. Een aparte
drempel van 5 tilt de gedeelde schaal omhoog zonder het uniforme-schaal-
invariant los te laten.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
brenno closed this pull request 2026-08-06 00:17:00 +00:00
All checks were successful
scans / scans (pull_request) Successful in 2m40s
Required
Details
static-gate / static-gate (pull_request) Successful in 5m2s
Required
Details

Pull request closed

Sign in to join this conversation.
No description provided.