fix(callouts): stage in echte slot-aspectratio + alle verwijzingen tonen (#1854) #1876

Merged

brenno merged 3 commits from fix/callout-stage-aspect-1854 into main

2026-08-31 09:15:11 +00:00

brenno commented

2026-08-31 09:07:16 +00:00

Owner

Summary

#1854: de callout-editor-stage gebruikt nu de aspectratio van het echte beeldslot (imgFraction * 16/9, ~0.71 bij default imageSize 40) in plaats van de dialoogverhouding (~1.42). Daardoor dekt cover hetzelfde deel als op de dia.
#1854: alle callouts tonen in de editor; de geselecteerde is interactief, de overige zijn gedimde statische markeringen. Het beeld is altijd zichtbaar, ook zonder selectie.
#1854: de bijsnijddialoog toont callout-markeringen mee, zodat de auteur ziet welke doelen al geplaatst zijn en of bijsnijden ze uit beeld schuift.
Statische markeringen staan in lib/widgets/editors/callout_marker_helpers.dart hergebruikbaar.
Regressietests voor alle drie fixes toegevoegd.
Orphan-l10n-string "Selecteer een regel om een verwijzing te plaatsen." verwijderd uit alle 31 vertaalbestanden.
Conventie-ceilings bewust verhoogd met reden (file + class).

Test plan

make check groen (format, analyzer, conventions, full test-suite, coverage floor).
make check-secrets groen.
make sast groen.

Generated with Devin

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

## Summary - #1854: de callout-editor-stage gebruikt nu de aspectratio van het echte beeldslot (`imgFraction * 16/9`, ~0.71 bij default imageSize 40) in plaats van de dialoogverhouding (~1.42). Daardoor dekt `cover` hetzelfde deel als op de dia. - #1854: alle callouts tonen in de editor; de geselecteerde is interactief, de overige zijn gedimde statische markeringen. Het beeld is altijd zichtbaar, ook zonder selectie. - #1854: de bijsnijddialoog toont callout-markeringen mee, zodat de auteur ziet welke doelen al geplaatst zijn en of bijsnijden ze uit beeld schuift. - Statische markeringen staan in `lib/widgets/editors/callout_marker_helpers.dart` hergebruikbaar. - Regressietests voor alle drie fixes toegevoegd. - Orphan-l10n-string "Selecteer een regel om een verwijzing te plaatsen." verwijderd uit alle 31 vertaalbestanden. - Conventie-ceilings bewust verhoogd met reden (file + class). ## Test plan - `make check` groen (format, analyzer, conventions, full test-suite, coverage floor). - `make check-secrets` groen. - `make sast` groen. Generated with [Devin](https://devin.ai) Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno added 3 commits

2026-08-31 09:07:16 +00:00

fix(callouts): stage in echte slot-aspectratio + alle verwijzingen tonen (#1854 ) 9b5c52bb45

De callout-editor gebruikte de dialoogverhouding (~1,42) in plaats van de
beeldslot-verhouding (~0,71), waardoor cover een ander deel van de afbeelding
afdekte dan op de dia. Nu staat de stage in een Center+AspectRatio met
imgFraction*16/9, net als bullets_image_editor aan de bijsnijddialoog
doorgeeft.

Daarnaast toonde de editor alleen de geselecteerde verwijzing; de overige
waren onzichtbaar. Nu doorloopt _buildImageStack alle callouts en toont de
niet-geselecteerde als gedimde statische markeringen (buildStaticCalloutMarkers
in callout_marker_helpers.dart). Het beeld is altijd aanwezig, ook zonder
selectie — voorheen toonde het werkvlak "Selecteer een regel".

De bijsnijddialoog toont nu ook de callout-markeringen, zodat de auteur ziet
welke doelen al geplaatst zijn en of bijsnijden ze uit beeld schuift.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

test(callouts): regressietests voor slot-aspectratio, alle verwijzingen en crop-markeringen (#1854 ) 4b224f3df7

Drie tests voor de callout-editor: de stage gebruikt de slot-aspectratio
(AspectRatio-widget met imgFraction*16/9), alle verwijzingen tonen zonder
selectie (zowel A als B zichtbaar), en het beeld is altijd aanwezig ook
zonder selectie. Eén test voor de bijsnijddialoog: callout-markeringen tonen
in de stage.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

chore(callouts): verwijder orphan-string + verhoog ceiling voor #1854

scans / scans (pull_request) Successful in 3m7s

Details

static-gate / static-gate (pull_request) Successful in 7m5s

Details

cc0634b459

De string "Selecteer een regel om een verwijzing te plaatsen." is uit alle
31 vertaalbestanden verwijderd — het beeld is nu altijd zichtbaar en de
tekst wordt nergens meer aangeroepen.

De file-ceiling voor callout_editor.dart gaat van 1045 naar 1056 en de
klasse-ceiling voor _CalloutEditorDialogState van 1000 naar 1009, beide met
reden: de null-image guard en de for-loop die alle callouts doorloopt.

Generated with [Devin](https://devin.ai)

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>

brenno merged commit 737a364dcc into main

2026-08-31 09:15:11 +00:00

brenno referenced this pull request from a commit

2026-08-31 09:15:13 +00:00

fix(callouts): stage in echte slot-aspectratio + alle verwijzingen tonen (#1854) (#1876)