refactor: verwijder enableZoom-parameter uit bijsnijddialoog (#1831) #1838

Merged

brenno merged 1 commit from fix/remove-enable-zoom-1831 into main

2026-08-29 08:38:16 +00:00

brenno commented

2026-08-29 08:31:19 +00:00

Owner

Samenvatting

Alle vijf de aanroepers van showImageCropDialog gaven enableZoom: true mee; de false-stand bestond alleen in de test-helper open(), waar negen toetsen hem als standaard lieten staan. Dat gaf groene dekking op een productiestand die niemand kan bereiken — _cover, de pinch-guard en de schuif-voorwaarde gedroegen zich anders dan in productie.

Route 2 gekozen (uit het issue): de parameter gaat weg.

_cover wordt _size == 0 (imageSize 0 = vullend/cover, anders zoompercentage)
pinch-guard wordt s.scale != 1.0 (was widget.enableZoom && s.scale != 1.0)
schuif-voorwaarde wordt !_cover (was widget.enableZoom && !_cover)
enableZoom: true verwijderd uit 5 aanroepers
dartdoc bijgewerkt

De toets "zonder zoom blijft de opgegeven maat ongemoeid" gaat mee weg: zijn commentaar beweerde dat imageSize in een kolomslot de kolombreedte is — precies het verhaal dat #1813 uit de dartdoc heeft gehaald omdat het niet klopte. Met route 2 bestaat die stand niet meer.

Bewaker

Expliciet overgeslagen: deze wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer, of publieke belofte. Het verwijdert een interne parameter uit een widget-dialoog.

Test plan

make check groen (10813 tests, 86.8% coverage)
make check-secrets schoon (gitleaks + trufflehog, 0 leaks)
make sast schoon (semgrep, 0 findings)
Geen enableZoom-referenties meer in de codebase
flutter analyze --fatal-infos schoon

Generated with Devin

## Samenvatting Alle vijf de aanroepers van `showImageCropDialog` gaven `enableZoom: true` mee; de `false`-stand bestond alleen in de test-helper `open()`, waar negen toetsen hem als standaard lieten staan. Dat gaf groene dekking op een productiestand die niemand kan bereiken — `_cover`, de pinch-guard en de schuif-voorwaarde gedroegen zich anders dan in productie. **Route 2 gekozen** (uit het issue): de parameter gaat weg. - `_cover` wordt `_size == 0` (imageSize 0 = vullend/cover, anders zoompercentage) - pinch-guard wordt `s.scale != 1.0` (was `widget.enableZoom && s.scale != 1.0`) - schuif-voorwaarde wordt `!_cover` (was `widget.enableZoom && !_cover`) - `enableZoom: true` verwijderd uit 5 aanroepers - dartdoc bijgewerkt De toets "zonder zoom blijft de opgegeven maat ongemoeid" gaat mee weg: zijn commentaar beweerde dat `imageSize` in een kolomslot de kolombreedte is — precies het verhaal dat #1813 uit de dartdoc heeft gehaald omdat het niet klopte. Met route 2 bestaat die stand niet meer. ## Bewaker Expliciet overgeslagen: deze wijziging raakt geen bestandsformaat, opslag, afhankelijkheid, uitgaand verkeer, of publieke belofte. Het verwijdert een interne parameter uit een widget-dialoog. #### Test plan - [x] `make check` groen (10813 tests, 86.8% coverage) - [x] `make check-secrets` schoon (gitleaks + trufflehog, 0 leaks) - [x] `make sast` schoon (semgrep, 0 findings) - [x] Geen `enableZoom`-referenties meer in de codebase - [x] `flutter analyze --fatal-infos` schoon Generated with [Devin](https://devin.ai)

brenno added 1 commit

2026-08-29 08:31:19 +00:00

refactor: verwijder enableZoom-parameter uit bijsnijddialoog (#1831 )

scans / scans (pull_request) Successful in 2m28s

Details

static-gate / static-gate (pull_request) Successful in 6m16s

Details

91dfff78c2

Alle vijf de aanroepers gaven enableZoom: true mee; de false-stand
bestond alleen in de test-helper, waar negen toetsen hem als standaard
lieten staan. Dat gaf groene dekking op een productiestand die niemand
kan bereiken — _cover, pinch-guard en schuif-voorwaarde gedroegen zich
anders dan in productie.

De parameter gaat weg (route 2 uit het issue): _cover wordt _size == 0,
de pinch-guard wordt s.scale != 1.0, en de schuif-voorwaarde wordt
!_cover. Kleiner oppervlak, eerlijker over wat het venster kan.

De toets "zonder zoom blijft de opgegeven maat ongemoeid" gaat mee weg:
zijn commentaar beweerde dat imageSize in een kolomslot de kolombreedte
is, precies het verhaal dat #1813 uit de dartdoc heeft gehaald omdat het
niet klopte.

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

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

brenno merged commit 934f57ea08 into main

2026-08-29 08:38:16 +00:00

brenno referenced this pull request from a commit

2026-08-29 08:38:18 +00:00

refactor: verwijder enableZoom-parameter uit bijsnijddialoog (#1831) (#1838)