feat(shell): geef de presentatie-eigenschappen-knop een label (#1479) #1484

Merged

brenno merged 2 commits from feat/presentation-properties-button into main

2026-08-13 20:42:36 +00:00

brenno commented

2026-08-13 20:06:06 +00:00

Owner

Wat & waarom (#1479)

De globale presentatie-instellingen zaten achter een klein rondje-"i" in de AppBar-titel. De melder (kwoot) gaf aan dat dat te klein is voor iets wat de metadata én het afspeelgedrag van de hele presentatie regelt, en dat een info-"i" niet uitnodigt — zo'n "i" leest als "hier valt iets te lézen", terwijl je er juist instelt.

Oplossing

Het losse IconButton (Icons.info_outline, size 18) is vervangen door een zichtbaar gelabelde knop "Presentatie-eigenschappen" met een instel-icoon (Icons.tune), gestyled als de naastgelegen TLP-chip (rand, afronding, _onAppBar-kleur zodat hij ook op een licht profiel leesbaar blijft, #780).
Het label hergebruikt de bestaande vertaalde string Presentatie-eigenschappen (de titel van dezelfde dialoog) — alle 31 talen waren al gedekt, geen nieuwe l10n.
Het commandopalet gebruikt nu hetzelfde Icons.tune voor "Eigenschappen", zodat beide ingangen naar dezelfde dialoog er hetzelfde uitzien.
De verweesde t()-sleutel presentationProperties (enige gebruiker was de oude knop) is uit alle 32 taalbestanden verwijderd.

Verificatie

flutter analyze (gewijzigde bestanden): schoon.
make l10n-check: groen (23 tests).
dart format: schoon.
Visueel gekeurd (licht + donker thema) vóór merge.

Sluit #1479.

## Wat & waarom (#1479) De globale presentatie-instellingen zaten achter een klein rondje-"i" in de AppBar-titel. De melder (kwoot) gaf aan dat dat te klein is voor iets wat de metadata én het afspeelgedrag van de hele presentatie regelt, en dat een info-"i" niet uitnodigt — zo'n "i" leest als "hier valt iets te lézen", terwijl je er juist instelt. ## Oplossing - Het losse `IconButton` (`Icons.info_outline`, size 18) is vervangen door een zichtbaar gelabelde knop **"Presentatie-eigenschappen"** met een instel-icoon (`Icons.tune`), gestyled als de naastgelegen TLP-chip (rand, afronding, `_onAppBar`-kleur zodat hij ook op een licht profiel leesbaar blijft, #780). - Het label hergebruikt de bestaande vertaalde string `Presentatie-eigenschappen` (de titel van dezelfde dialoog) — alle 31 talen waren al gedekt, geen nieuwe l10n. - Het commandopalet gebruikt nu hetzelfde `Icons.tune` voor "Eigenschappen", zodat beide ingangen naar dezelfde dialoog er hetzelfde uitzien. - De verweesde `t()`-sleutel `presentationProperties` (enige gebruiker was de oude knop) is uit alle 32 taalbestanden verwijderd. ## Verificatie - `flutter analyze` (gewijzigde bestanden): schoon. - `make l10n-check`: groen (23 tests). - `dart format`: schoon. - Visueel gekeurd (licht + donker thema) vóór merge. Sluit #1479.

brenno added 1 commit

2026-08-13 20:06:06 +00:00

feat(shell): geef de presentatie-eigenschappen-knop een label (#1479 )

scans / scans (pull_request) Successful in 3m56s

Details

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

Details

2654dc71fa

De globale presentatie-instellingen zaten achter een klein rondje-"i" in
de AppBar-titel. Een info-"i" leest als "hier valt iets te lézen", terwijl
je in deze dialoog juist de metadata én het afspeelgedrag van de hele
presentatie instelt; op grootte 18 was hij bovendien te klein voor dat
gewicht.

Vervangen door een zichtbaar gelabelde knop ("Presentatie-eigenschappen",
tune-icoon), gestyled als de naastgelegen TLP-chip en kleur-adaptief via
_onAppBar. Hergebruikt de bestaande dialoogtitel-string, al gedekt in alle
31 talen. Het commandopalet gebruikt nu hetzelfde tune-icoon voor
"Eigenschappen", en de verweesde t()-sleutel `presentationProperties` is
uit alle taalbestanden verwijderd.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno added 1 commit

2026-08-13 20:34:59 +00:00

test(shell): tik op de gelabelde eigenschappen-knop, niet op het "i"-icoon (#1479 )

scans / scans (pull_request) Successful in 3m25s

Details

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

Details

a63c512c05

De AppBar-knop draagt sinds de vorige commit een zichtbaar label in plaats
van `Icons.info_outline`; deze shell-test zocht nog naar dat verdwenen icoon.
De test tikt nu op de gelabelde knop en bevestigt bovendien dat het label
"Presentatie-eigenschappen" in de AppBar-titel staat.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit d7c2a06ea2 into main

2026-08-13 20:42:36 +00:00

brenno deleted branch feat/presentation-properties-button

2026-08-13 20:42:37 +00:00

brenno referenced this pull request from a commit

2026-08-13 20:42:38 +00:00

Merge pull request 'feat(shell): geef de presentatie-eigenschappen-knop een label (#1479)' (#1484) from feat/presentation-properties-button into main

brenno referenced this pull request

2026-08-13 20:43:34 +00:00

[Feature] rename small "i" in circle to "Presentation settings" #1479