fix(overzicht): neerzetten op expliciete invoegposities (#2362) #2371

Merged
brenno merged 2 commits from fix/overview-insert-positions into main 2026-10-09 00:17:09 +00:00
Owner

Samenvatting

  • Dropbestemmingen in het slide-overzicht zijn nu invoegslots 0..N in plaats van kaartdoelen: vóór de eerste dia, tussen elke twee dia's en achter de laatste via een eigen "Achteraan plaatsen"-doelzone.
  • Tijdens het slepen toont elke slotrand een subtiele streep en de actieve positie een accentlijn.
  • Verplaatsing gecentraliseerd in moveSlidesToSlot; reorderSlides/moveSlides vertalen daarheen.
  • Slots in of direct langs het gesleepte blok zijn een no-op; een meervoudige selectie verhuist als één blok in eigen volgorde, met selectie en primaire selectie mee.
  • Drie nieuwe l10n-strings in alle 30 talen; golden slide_overview_narrow_insert_marker legt de markering vast.
  • Bonusfix: image_carousel_download_test wachtte op bestaan i.p.v. inhoud van de kopie — race onder load.

Test plan

  • make check volledig groen (format, analyze, conventies, suite, coverage, goldens)
  • Widget-tests: slepen naar slot, slepen naar eindzone, meervoudige selectie als blok
  • Golden gecontroleerd: accentlijn, hintstrepen en eindzone zichtbaar
  • Live-controle in draaiende app niet mogelijk geweest (geen computer-use in deze sessie)

Generated with Devin

## Samenvatting - Dropbestemmingen in het slide-overzicht zijn nu invoegslots 0..N in plaats van kaartdoelen: vóór de eerste dia, tussen elke twee dia's en achter de laatste via een eigen "Achteraan plaatsen"-doelzone. - Tijdens het slepen toont elke slotrand een subtiele streep en de actieve positie een accentlijn. - Verplaatsing gecentraliseerd in `moveSlidesToSlot`; `reorderSlides`/`moveSlides` vertalen daarheen. - Slots in of direct langs het gesleepte blok zijn een no-op; een meervoudige selectie verhuist als één blok in eigen volgorde, met selectie en primaire selectie mee. - Drie nieuwe l10n-strings in alle 30 talen; golden `slide_overview_narrow_insert_marker` legt de markering vast. - Bonusfix: `image_carousel_download_test` wachtte op bestaan i.p.v. inhoud van de kopie — race onder load. #### Test plan - [x] `make check` volledig groen (format, analyze, conventies, suite, coverage, goldens) - [x] Widget-tests: slepen naar slot, slepen naar eindzone, meervoudige selectie als blok - [x] Golden gecontroleerd: accentlijn, hintstrepen en eindzone zichtbaar - [ ] Live-controle in draaiende app niet mogelijk geweest (geen computer-use in deze sessie) Generated with [Devin](https://devin.ai)
File.copy maakt het doelbestand aan zodra het voor schrijven opent, dus
existsSync() kan true geven voordat de bytes er staan. Onder load sloeg
de lees-direct-na-bestaan-check dan op een leeg bestand.

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

Co-Authored-By: Devin <158243242+devin-ai-integration[bot]@users.noreply.github.com>
Dropbestemmingen in het slide-overzicht zijn nu invoegslots 0..N in
plaats van kaartdoelen: vóór de eerste dia, tussen elke twee dia's en
achter de laatste via een eigen doelzone. Tijdens het slepen toont elke
slotrand een subtiele streep en de actieve positie een accentlijn.

De verplaatsing zelf zit gecentraliseerd in moveSlidesToSlot; de oude
index-gebaseerde reorder/moveSlides vertalen daarheen. Slots in of
direct langs het gesleepte blok zijn een no-op, en een geselecteerd blok
verhuist als geheel in eigen volgorde met selectie en primaire selectie
mee.

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

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