fix(crop): zoomregelaar bereikbaar met muis/toetsenbord + thema-achtergrond (#1856) #1869

Merged
brenno merged 1 commit from fix/crop-zoom-accessibility-1856 into main 2026-08-30 09:48:17 +00:00
Owner

Samenvatting

  • Zoomregelaar altijd zichtbaar: de schuif stond achter if (!_cover) en was alleen met een pinch-gebaar (twee vingers) te bereiken. Nu staat hij altijd in beeld — in cover-modus toont hij op minZoom, en slepen verlaat cover en stelt een echte zoom in.
  • Herstel reset ook zoom: de Herstel-knop zet nu ook _size terug naar de oorspronkelijke waarde, niet alleen brandpunt en rotatie.
  • Letterbox volgt thema: de stage-achtergrond was hardcoded Colors.black. Bij zoom > 100% waren de letterbox-balken hierdoor zuiver zwart in elk thema. Valt nu terug op Theme.of(context).colorScheme.surface.

Test plan

  • make check groen (10968 tests, 87.0% coverage)
  • make check-secrets schoon (gitleaks + trufflehog)
  • make sast schoon (semgrep, 0 findings)
  • Nieuwe tests: schuif bereikbaar in cover-modus, schuif verlaat cover, Herstel reset zoom, achtergrond niet zuiver zwart

Generated with Devin

## Samenvatting - **Zoomregelaar altijd zichtbaar**: de schuif stond achter `if (!_cover)` en was alleen met een pinch-gebaar (twee vingers) te bereiken. Nu staat hij altijd in beeld — in cover-modus toont hij op minZoom, en slepen verlaat cover en stelt een echte zoom in. - **Herstel reset ook zoom**: de Herstel-knop zet nu ook `_size` terug naar de oorspronkelijke waarde, niet alleen brandpunt en rotatie. - **Letterbox volgt thema**: de stage-achtergrond was hardcoded `Colors.black`. Bij zoom > 100% waren de letterbox-balken hierdoor zuiver zwart in elk thema. Valt nu terug op `Theme.of(context).colorScheme.surface`. #### Test plan - [x] `make check` groen (10968 tests, 87.0% coverage) - [x] `make check-secrets` schoon (gitleaks + trufflehog) - [x] `make sast` schoon (semgrep, 0 findings) - [x] Nieuwe tests: schuif bereikbaar in cover-modus, schuif verlaat cover, Herstel reset zoom, achtergrond niet zuiver zwart Generated with [Devin](https://devin.ai)
fix(crop): zoomregelaar bereikbaar met muis/toetsenbord + thema-achtergrond (#1856)
All checks were successful
scans / scans (pull_request) Successful in 3m39s
static-gate / static-gate (pull_request) Successful in 8m35s
3ea2b95e96
De zoomregelaar in de bijsnijddialoog stond achter if (!_cover), en de
enige weg uit cover was een pinch-gebaar — onbereikbaar met muis of
toetsenbord. De schuif staat nu altijd in beeld; in cover-modus toont
hij op minZoom en slepen verlaat cover. Herstel zet ook de zoom terug
naar de oorspronkelijke waarde.

De letterbox-balken bij zoom > 100% waren zuiver zwart in elk thema. De
stage-achtergrond valt nu terug op Theme.of(context).colorScheme.surface
in plaats van Colors.black, zodat de balken het thema volgen.

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

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
brenno merged commit 6dc3f5b8ba into main 2026-08-30 09:48:17 +00:00
Sign in to join this conversation.
No description provided.