fix(present): laat een mermaid-diagram uitzoomen tot het hele diagram past (#944) #946

Merged

brenno merged 1 commit from fix/mermaid-zoom-out into main

2026-07-28 02:29:07 +00:00

brenno commented

2026-07-28 02:18:30 +00:00

Owner

Verhelpt #944 (vervolg op #930).

De zoom kon niet ver genoeg uitzoomen: je kon inzoomen maar niet uitzoomen om het hele diagram te zien. Twee dingen hielden het tegen — mermaidViewMatrix klemde de zoomfactor op ≥ 1, en InteractiveViewer klemt met boundaryMargin: EdgeInsets.zero de min-zoom stil terug op "diagram vult nog het venster". Beide losgelaten; de grenzen bewaakt de widget nu zelf (mermaidClampView).

De ondergrens is nu de passend-in-het-venster-factor (mermaidFitScale, < 1 voor een overlopend diagram): knijpen en de min-knop zoomen uit tot het hele diagram past, en de passend-knop springt daar in één klik naartoe. Maat-onafhankelijk, dus spiegelt mee naar de beamer.

Test. De toets meet nu de gerénderde maat (getRect), niet slechts de opgeslagen zoom — dát was het gat waardoor de vorige poging groen was terwijl de app niet uitzoomde. make check groen.

Draagt ook een dart format-normalisatie van test/native_git_mirror_test.dart mee: die stond ongeformatteerd op main (een gelijktijdige merge), waardoor de format-poort er anders op zou vallen.

Verhelpt #944 (vervolg op #930). De zoom kon niet ver genoeg uitzoomen: je kon inzoomen maar niet uitzoomen om het hele diagram te zien. Twee dingen hielden het tegen — `mermaidViewMatrix` klemde de zoomfactor op ≥ 1, en `InteractiveViewer` klemt met `boundaryMargin: EdgeInsets.zero` de min-zoom stil terug op "diagram vult nog het venster". Beide losgelaten; de grenzen bewaakt de widget nu zelf (`mermaidClampView`). De ondergrens is nu de passend-in-het-venster-factor (`mermaidFitScale`, < 1 voor een overlopend diagram): knijpen en de min-knop zoomen uit tot het hele diagram past, en de passend-knop springt daar in één klik naartoe. Maat-onafhankelijk, dus spiegelt mee naar de beamer. **Test.** De toets meet nu de *gerénderde* maat (`getRect`), niet slechts de opgeslagen zoom — dát was het gat waardoor de vorige poging groen was terwijl de app niet uitzoomde. `make check` groen. Draagt ook een `dart format`-normalisatie van `test/native_git_mirror_test.dart` mee: die stond ongeformatteerd op main (een gelijktijdige merge), waardoor de format-poort er anders op zou vallen.

brenno force-pushed fix/mermaid-zoom-out from 1f72e239e4

scans / scans (pull_request) Successful in 3m21s

Details

to 1ea8c46a46

scans / scans (pull_request) Successful in 3m31s

Details

2026-07-28 02:25:31 +00:00

Compare

brenno merged commit 482f421158 into main

2026-07-28 02:29:07 +00:00

brenno deleted branch fix/mermaid-zoom-out

2026-07-28 02:29:07 +00:00