fix(present): reserveer de echte hoogte van een LaTeX-formule bij paginering #947

Merged

brenno merged 1 commit from fix/math-block-height-logo into main

2026-07-28 10:57:57 +00:00

brenno commented

2026-07-28 10:56:51 +00:00

Owner

Dia 34 (“De kat van Schrödinger”) liet bij presentatiemaat de onderste regel over het logo vallen. Oorzaak: de richText-paginering mat een $$…$$-blok als één tekstregel, terwijl Math.tex met een breuk/wortel 2á3× zo hoog rendert (gemeten 27 en 43px te weinig). displayMathBlockHeight schat de hoogte nu inhoud-bewust; de dia klaart het logo. Tegenhanger van #931 (free-markdown/FittedBox); dit is het richText/paginerings-pad.

Regressietest math_block_height_test ijkt elke schatting tegen de echte Math.tex-hoogte (rood tegen de oude schatting). Live gereproduceerd in de app (LibreKAT/EB Garamond) en na de fix bevestigd.

Poort: volledige make check groen vóór de rebase; na de rebase op de laatste main gericht herdraaid (format, analyze, betrokken tests — alle groen). Secrets/SAST-scan (make check-full) niet volledig herdraaid wegens tijd; wijziging is pure Dart-layoutlogica zonder nieuwe afhankelijkheden, geheimen of netwerk.

Bewaker: n.v.t. — raakt bestandsformaat, opslag, afhankelijkheden, uitgaand verkeer noch een publieke belofte.

Dia 34 (“De kat van Schrödinger”) liet bij presentatiemaat de onderste regel over het logo vallen. Oorzaak: de richText-paginering mat een `$$…$$`-blok als één tekstregel, terwijl `Math.tex` met een breuk/wortel 2á3× zo hoog rendert (gemeten 27 en 43px te weinig). `displayMathBlockHeight` schat de hoogte nu inhoud-bewust; de dia klaart het logo. Tegenhanger van #931 (free-markdown/FittedBox); dit is het richText/paginerings-pad. Regressietest `math_block_height_test` ijkt elke schatting tegen de echte `Math.tex`-hoogte (rood tegen de oude schatting). Live gereproduceerd in de app (LibreKAT/EB Garamond) en na de fix bevestigd. Poort: volledige `make check` groen vóór de rebase; na de rebase op de laatste main gericht herdraaid (format, analyze, betrokken tests — alle groen). Secrets/SAST-scan (`make check-full`) niet volledig herdraaid wegens tijd; wijziging is pure Dart-layoutlogica zonder nieuwe afhankelijkheden, geheimen of netwerk. Bewaker: n.v.t. — raakt bestandsformaat, opslag, afhankelijkheden, uitgaand verkeer noch een publieke belofte.

brenno added 1 commit

2026-07-28 10:56:51 +00:00

fix(present): reserveer de echte hoogte van een LaTeX-formule bij paginering

scans / scans (pull_request) Has been cancelled

Details

c0671e61e6

Een `$$…$$`-blok op een bullets/richText-dia werd door de paginering als één
tekstregel gemeten (`refW * 0.032` + padding), terwijl `Math.tex` op die vaste
grootte rendert maar met een breukstreep, wortel, grote operator of gestapelde
exponent twee à drie keer zo hoog is. De plan dacht dat de inhoud paste, de
render werd ~70px hoger (27 + 43px voor de twee "kat van Schrödinger"-formules),
en de onderste regel zakte bij presentatiemaat in de logostrook — empirisch
bevestigd in de LibreKAT-stijl (EB Garamond); in het compactere Arial paste het
nog nét, wat de reproductie eerder verborg.

`displayMathBlockHeight` schat de verticale omvang nu inhoud-bewust uit de
constructies die hoogte toevoegen, ruim aan de veilige kant: onderschatten raakt
het logo, overschatten pagineert of schaalt hooguit iets eerder. De dia klaart
het logo nu netjes. Tegenhanger van #931 (dat het free-markdown/FittedBox-pad
raakte); dit is het richText/paginerings-pad.

Regressietest: `math_block_height_test` ijkt elke schatting tegen de werkelijke
`Math.tex`-hoogte van een batterij formules, langs het volledige parse→meet-pad
dat de paginering gebruikt — rood tegen de oude één-regel-schatting.

Bewaker: n.v.t. — puur een render-/layoutmeting; raakt bestandsformaat, opslag,
afhankelijkheden, uitgaand verkeer noch een publieke belofte niet.

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

brenno force-pushed fix/math-block-height-logo from c0671e61e6

scans / scans (pull_request) Has been cancelled

Details

to b9be786040

scans / scans (pull_request) Successful in 3m18s

Details

2026-07-28 10:57:44 +00:00

Compare

brenno merged commit a09e7e7a92 into main

2026-07-28 10:57:57 +00:00

brenno referenced this pull request from a commit

2026-07-28 10:57:57 +00:00