fix(callouts): plaatsingsmodus, leesvolgorde-letters, presentatiehint (#1860) #1878

Merged

brenno merged 1 commit from fix/callout-onboarding-1860 into main

2026-08-31 11:46:05 +00:00

brenno commented

2026-08-31 11:05:14 +00:00

Owner

Samenvatting

Issue #1860 beschrijft vier UX-problemen in de verwijzingendialoog, gemeld door een nieuwe gebruiker. Deze PR lost alle vier op.

1. Click-to-place (punten 1 + 2)

"Toevoegen" plaatst niet meer meteen een marker op 0.5/0.5 zonder instructie. In plaats daarvan gaat de editor in plaatsingsmodus: de callout wordt aangemaakt (met letter op de bullet) maar krijgt pas een doel als de gebruiker op het beeld klikt (pin/pijl) of sleept (gebied). Een instructie-overlay toont "Klik op de afbeelding waar deze regel naar verwijst" of "Sleep op de afbeelding om een gebied te markeren".

Een tweede "Toevoegen" annuleert de onvoltooide plaatsing (herstel ontbrak volgens het issue).
Incomplete callouts (zonder doel) worden niet geëmit naar de dia.
In plaatsingsmodus staat de gesture layer bovenop de markers, zodat een tik altijd het nieuwe doel plaatst.
"Doel toevoegen" is uitgeschakeld in plaatsingsmodus.

2. Presentatiewijze-feedback (punt 3)

Onder de SegmentedButton voor presentatiewijze staat nu een hint: "Geldt voor de hele dia — bepaalt hoe verwijzingen tijdens de presentatie worden getekend, niet hoe je ze bewerkt." Dit legt uit dat de keuze dia-breed is en alleen de presentatie beïnvloedt.

3. Letters in leesvolgorde (punt 4)

_renumberReferences herschikt bij elke emit de callouts in leesvolgorde (boven naar beneden) en wijst letters A, B, C, … toe. Voorheen volgden letters de aanmaakvolgorde — het publiek zag A, B, D, C als je verwijzingen uit die volgorde aanmaakte. De front matter schrijft callouts nu ook in leesvolgorde.

Bewaker-toets

De wijziging raakt wat er in het .md terechtkomt: callouts staan in de front matter nu in leesvolgorde in plaats van aanmaakvolgorde. Het formaat (velden, structuur) is ongewijzigd — alleen de volgorde verandert. Dit botst niet met de kernwaarden:

Uitwisselbaarheid: een ander Marp-gereedschap leest dezelfde data, alleen in een intuïtievere volgorde.
Soevereiniteit: geen nieuwe partij, geen lock-in.
De scherpe toets: als OciDeck ophoudt, blijven de decks leesbaar.

Tests

5 bestaande tests aangepast voor het nieuwe click-to-place gedrag (plaatsingsstap toegevoegd).
5 nieuwe tests: plaatsingsmodus toont instructie, gebied-modus toont sleep-instructie, tweede Toevoegen annuleert onvoltooide plaatsing, letters hernummeren in leesvolgorde, presentatiewijze toont hint.
make check groen: 11005 tests, 87% dekking.
make check-secrets groen: gitleaks + trufflehog, 0 bevindingen.
make sast groen: semgrep, 0 bevindingen.

Test plan

make check — 11005 tests geslaagd
make check-secrets — gitleaks + trufflehog schoon
make sast — semgrep schoon
make l10n-check — alle 31 talen gedekt
CI static-gate groen

Generated with Devin

## Samenvatting Issue #1860 beschrijft vier UX-problemen in de verwijzingendialoog, gemeld door een nieuwe gebruiker. Deze PR lost alle vier op. ### 1. Click-to-place (punten 1 + 2) "Toevoegen" plaatst niet meer meteen een marker op 0.5/0.5 zonder instructie. In plaats daarvan gaat de editor in **plaatsingsmodus**: de callout wordt aangemaakt (met letter op de bullet) maar krijgt pas een doel als de gebruiker op het beeld klikt (pin/pijl) of sleept (gebied). Een instructie-overlay toont "Klik op de afbeelding waar deze regel naar verwijst" of "Sleep op de afbeelding om een gebied te markeren". - Een tweede "Toevoegen" annuleert de onvoltooide plaatsing (herstel ontbrak volgens het issue). - Incomplete callouts (zonder doel) worden niet geëmit naar de dia. - In plaatsingsmodus staat de gesture layer bovenop de markers, zodat een tik altijd het nieuwe doel plaatst. - "Doel toevoegen" is uitgeschakeld in plaatsingsmodus. ### 2. Presentatiewijze-feedback (punt 3) Onder de SegmentedButton voor presentatiewijze staat nu een hint: "Geldt voor de hele dia — bepaalt hoe verwijzingen tijdens de presentatie worden getekend, niet hoe je ze bewerkt." Dit legt uit dat de keuze dia-breed is en alleen de presentatie beïnvloedt. ### 3. Letters in leesvolgorde (punt 4) `_renumberReferences` herschikt bij elke emit de callouts in leesvolgorde (boven naar beneden) en wijst letters A, B, C, … toe. Voorheen volgden letters de aanmaakvolgorde — het publiek zag A, B, D, C als je verwijzingen uit die volgorde aanmaakte. De front matter schrijft callouts nu ook in leesvolgorde. ### Bewaker-toets De wijziging raakt wat er in het `.md` terechtkomt: callouts staan in de front matter nu in leesvolgorde in plaats van aanmaakvolgorde. Het formaat (velden, structuur) is ongewijzigd — alleen de volgorde verandert. Dit botst niet met de kernwaarden: - **Uitwisselbaarheid**: een ander Marp-gereedschap leest dezelfde data, alleen in een intuïtievere volgorde. - **Soevereiniteit**: geen nieuwe partij, geen lock-in. - De scherpe toets: als OciDeck ophoudt, blijven de decks leesbaar. ## Tests - 5 bestaande tests aangepast voor het nieuwe click-to-place gedrag (plaatsingsstap toegevoegd). - 5 nieuwe tests: plaatsingsmodus toont instructie, gebied-modus toont sleep-instructie, tweede Toevoegen annuleert onvoltooide plaatsing, letters hernummeren in leesvolgorde, presentatiewijze toont hint. - `make check` groen: 11005 tests, 87% dekking. - `make check-secrets` groen: gitleaks + trufflehog, 0 bevindingen. - `make sast` groen: semgrep, 0 bevindingen. #### Test plan - [x] `make check` — 11005 tests geslaagd - [x] `make check-secrets` — gitleaks + trufflehog schoon - [x] `make sast` — semgrep schoon - [x] `make l10n-check` — alle 31 talen gedekt - [ ] CI `static-gate` groen Generated with [Devin](https://devin.ai)

brenno added 1 commit

2026-08-31 11:05:14 +00:00

fix(callouts): plaatsingsmodus, leesvolgorde-letters, presentatiehint (#1860 )

scans / scans (pull_request) Successful in 2m19s

Details

static-gate / static-gate (pull_request) Successful in 5m42s

Details

1ec5a78212

De verwijzingendialoog plaatste meteen een marker op 0.5/0.5 zonder
instructie, de presentatiewijze gaf geen terugkoppeling, en letters
volgden aanmaakvolgorde in plaats van leesvolgorde. Vier wijzigingen:

1. "Toevoegen" gaat in plaatsingsmodus — toont "Klik op de afbeelding
   waar deze regel naar verwijst" (of "Sleep …" in gebied-modus) en
   plaatst het doel pas op de eerste klik/sleep. Een tweede "Toevoegen"
   annuleert de onvoltooide plaatsing. Incomplete callouts worden niet
   geëmit.

2. Onder de presentatiewijze-knoppen staat een hint: "Geldt voor de
   hele dia — bepaalt hoe verwijzingen tijdens de presentatie worden
   getekend, niet hoe je ze bewerkt."

3. _renumberReferences herschikt bij elke emit de callouts in
   leesvolgorde (boven naar beneden) en wijst letters A, B, C, … toe.
   De front-matter schrijft ze nu in leesvolgorde.

4. In plaatsingsmodus staat de gesture layer bovenop de markers zodat
   een tik altijd het nieuwe doel plaatst, niet een bestaande selecteert.

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

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

brenno merged commit 44f35ac8ac into main

2026-08-31 11:46:05 +00:00

brenno referenced this pull request from a commit

2026-08-31 11:46:08 +00:00

fix(callouts): plaatsingsmodus, leesvolgorde-letters, presentatiehint (#1860) (#1878)