fix(footer): paginanummer altijd rechtsonder, ook bij gecentreerde footer (#1330) #1333

Merged
brenno merged 1 commit from fix/page-numbers-bottom-right-1330 into main 2026-08-07 16:28:50 +00:00
Owner

De instelling "Paginanummers tonen (rechtsonder)" belooft de rechteronderhoek, maar het nummer zat in dezelfde Row als de footertekst en volgde daardoor de footerpositie. Bij Midden (de configuratie uit #1330) belandde het paginanummer in het midden naast de footertekst i.p.v. in de hoek.

Fix (lib/widgets/slides/previews/overlays.dart): het paginanummer krijgt een eigen plek rechtsonder, onafhankelijk van de footerpositie. Het wijkt — net als de footertekst al deed — horizontaal uit voor een logo of TLP-badge in die hoek, en de footertekst reserveert er ruimte voor zodat de twee elkaar nooit overlappen.

Klacht #2 uit het issue (spelerbalk overlapt de gecentreerde footer): visueel gekeurd in presentatiemodus. De zwevende, halftransparante spelerbalk staat grotendeels in de letterbox onder de 16:9-slide en bedekt de footertekst niet; belangrijker nog, deze fix haalt de vroegere botsing juist weg, want vóór de fix stond het paginanummer zélf in het midden — precies waar de balk verschijnt. Presentatie-chrome daarom ongewijzigd gelaten.

Verificatie

  • Regressietests in test/footer_preview_test.dart (paginanummer rechtsonder bij center/left, en zonder footertekst) — falen op de oude code, slagen op de nieuwe.
  • Volledige suite groen (8931 tests), coverage 87,1% (vloer 80%), alle statische poorten groen.
  • Beeldkeuring: editor-preview, fullscreen en PDF-export, licht én donker, met en zonder logo/TLP-badge — paginanummer netjes rechtsonder, geen overlap.

Closes #1330

De instelling **"Paginanummers tonen (rechtsonder)"** belooft de rechteronderhoek, maar het nummer zat in dezelfde `Row` als de footertekst en volgde daardoor de **footerpositie**. Bij *Midden* (de configuratie uit #1330) belandde het paginanummer in het midden naast de footertekst i.p.v. in de hoek. **Fix** (`lib/widgets/slides/previews/overlays.dart`): het paginanummer krijgt een eigen plek rechtsonder, onafhankelijk van de footerpositie. Het wijkt — net als de footertekst al deed — horizontaal uit voor een logo of TLP-badge in die hoek, en de footertekst reserveert er ruimte voor zodat de twee elkaar nooit overlappen. **Klacht #2 uit het issue** (spelerbalk overlapt de gecentreerde footer): visueel gekeurd in presentatiemodus. De zwevende, halftransparante spelerbalk staat grotendeels in de letterbox onder de 16:9-slide en bedekt de footertekst niet; belangrijker nog, deze fix haalt de vroegere botsing juist weg, want vóór de fix stond het paginanummer zélf in het midden — precies waar de balk verschijnt. Presentatie-chrome daarom ongewijzigd gelaten. **Verificatie** - Regressietests in `test/footer_preview_test.dart` (paginanummer rechtsonder bij center/left, en zonder footertekst) — falen op de oude code, slagen op de nieuwe. - Volledige suite groen (8931 tests), coverage 87,1% (vloer 80%), alle statische poorten groen. - Beeldkeuring: editor-preview, fullscreen en PDF-export, licht én donker, met en zonder logo/TLP-badge — paginanummer netjes rechtsonder, geen overlap. Closes #1330
brenno force-pushed fix/page-numbers-bottom-right-1330 from 8c14315ac7
All checks were successful
scans / scans (pull_request) Successful in 2m32s
static-gate / static-gate (pull_request) Successful in 7m19s
to b12e25fd9b
All checks were successful
scans / scans (pull_request) Successful in 1m39s
static-gate / static-gate (pull_request) Successful in 4m23s
2026-08-07 16:23:32 +00:00
Compare
brenno merged commit 13f136107a into main 2026-08-07 16:28:50 +00:00
Sign in to join this conversation.
No description provided.