Donker app-thema: Instellingen-knop op het openscherm niet leesbaar #916

New issue

Closed

opened 2026-07-27 09:36:43 +00:00 by brenno · 3 comments

brenno commented

2026-07-27 09:36:43 +00:00

Owner

Symptoom (melding gebruiker)

Met het app-uiterlijk op het ingebouwde profiel "Donker" werkt de hele interface goed, behálve het openscherm (welcome screen, zonder deck open): het stukje met de knop voor Instellingen onderaan is niet te lezen.

Stappen om te herhalen

Start OciDeck zonder een deck te openen (openscherm zichtbaar).
Zet het app-uiterlijk op profiel "Donker" (Instellingen → Uiterlijk).
Kijk naar de tekstlinks onderaan de middenkolom ("Gebruikershandleiding" en "Instellingen").

Verwacht: de Instellingen-link is net zo leesbaar als in het lichte profiel en als de rest van de app in Donker.

Nu: de Instellingen-sectie is onleesbaar in Donker.

Betrokken code: het openscherm is _WelcomeScreen in lib/widgets/shell/welcome_screen.dart (_footerLinks). Exacte oorzaak wordt visueel vastgesteld; de app-brede contrastsuite dekt dit geval kennelijk nog niet, dus er komt ook een regressietest bij.

**Symptoom (melding gebruiker)** Met het app-uiterlijk op het ingebouwde profiel "Donker" werkt de hele interface goed, behálve het openscherm (welcome screen, zonder deck open): het stukje met de knop voor **Instellingen** onderaan is niet te lezen. **Stappen om te herhalen** 1. Start OciDeck zonder een deck te openen (openscherm zichtbaar). 2. Zet het app-uiterlijk op profiel "Donker" (Instellingen → Uiterlijk). 3. Kijk naar de tekstlinks onderaan de middenkolom ("Gebruikershandleiding" en "Instellingen"). **Verwacht**: de Instellingen-link is net zo leesbaar als in het lichte profiel en als de rest van de app in Donker. **Nu**: de Instellingen-sectie is onleesbaar in Donker. Betrokken code: het openscherm is `_WelcomeScreen` in `lib/widgets/shell/welcome_screen.dart` (`_footerLinks`). Exacte oorzaak wordt visueel vastgesteld; de app-brede contrastsuite dekt dit geval kennelijk nog niet, dus er komt ook een regressietest bij.

brenno added the

bug

in-progress

labels

2026-07-27 09:36:43 +00:00

brenno commented

2026-07-27 09:37:10 +00:00

Author

Owner

Opgepakt op tak fix/916-donker-openscherm-instellingen.

Opgepakt op tak `fix/916-donker-openscherm-instellingen`.

brenno commented

2026-07-27 15:51:29 +00:00

Author

Owner

Live gereproduceerd én oorzaak gevonden — en die was al opgelost.

Reproductie (macOS, app-thema Donker, openscherm): de tekstlink Instellingen onderaan rendert donker (#111827, de merkkleur) op de donkere achtergrond #1E293B en is daardoor onleesbaar. Alle andere knoppen ernaast (omlijnde knoppen, de gevulde 'Nieuwe presentatie'-knop) waren wél wit/leesbaar — precies het beeld van één kale TextButton die op Materials standaard colorScheme.primary terugvalt.

Oorzaak = het ontbreken van een textButtonTheme-voorgrond. Dat is al gerepareerd door #744 (c3f681e6, 23-07-2026): textButtonTheme: TextButton.styleFrom(foregroundColor: profile.isDark ? text : primary) — in Donker dus #F1F5F9 (bijna-wit). De omlijnde-knop-regel bestond al, de tekstknop-regel ontbrak; #744 heeft die toegevoegd.

Waarom het tóch nog zichtbaar was: de geïnstalleerde /Applications/ocideck.app was een bouw van 07-07-2026, van vóór #744. De code op main is correct; er is een regressiewacht in test/app_theme_contrast_test.dart (groep 'de rollen die Material anders zelf invult, zijn expliciet gezet').

Geen code-wijziging op main nodig. Remedie voor de melder: de app opnieuw bouwen uit de huidige main (in uitvoering). Sluit ik als al-opgelost door #744 zodra de verse bouw dit bevestigt.

**Live gereproduceerd én oorzaak gevonden — en die was al opgelost.** Reproductie (macOS, app-thema Donker, openscherm): de tekstlink **Instellingen** onderaan rendert donker (`#111827`, de merkkleur) op de donkere achtergrond `#1E293B` en is daardoor onleesbaar. Alle andere knoppen ernaast (omlijnde knoppen, de gevulde 'Nieuwe presentatie'-knop) waren wél wit/leesbaar — precies het beeld van één kale `TextButton` die op Materials standaard `colorScheme.primary` terugvalt. Oorzaak = het ontbreken van een `textButtonTheme`-voorgrond. Dat is **al gerepareerd door #744** (`c3f681e6`, 23-07-2026): `textButtonTheme: TextButton.styleFrom(foregroundColor: profile.isDark ? text : primary)` — in Donker dus `#F1F5F9` (bijna-wit). De omlijnde-knop-regel bestond al, de tekstknop-regel ontbrak; #744 heeft die toegevoegd. Waarom het tóch nog zichtbaar was: de geïnstalleerde `/Applications/ocideck.app` was een bouw van **07-07-2026**, van vóór #744. De code op main is correct; er is een regressiewacht in `test/app_theme_contrast_test.dart` (groep 'de rollen die Material anders zelf invult, zijn expliciet gezet'). Geen code-wijziging op main nodig. Remedie voor de melder: de app opnieuw bouwen uit de huidige main (in uitvoering). Sluit ik als al-opgelost door #744 zodra de verse bouw dit bevestigt.

brenno commented

2026-07-27 16:04:39 +00:00

Author

Owner

Geverifieerd in een verse release-build van de huidige main (schoon: flutter clean → flutter build macos --release, App.framework-zegel gevalideerd) en geïnstalleerd in /Applications. In Donker is de Instellingen-link nu bijna-wit en leesbaar, net als de rest. Sluit als al-opgelost door #744; geen code-wijziging op main nodig.

Geverifieerd in een verse release-build van de huidige main (schoon: `flutter clean` → `flutter build macos --release`, App.framework-zegel gevalideerd) en geïnstalleerd in /Applications. In Donker is de Instellingen-link nu bijna-wit en leesbaar, net als de rest. Sluit als al-opgelost door #744; geen code-wijziging op main nodig.

brenno closed this issue

2026-07-27 16:04:39 +00:00