De donkere modus, ronde twee: de vijf oppervlakken die #780 niet heeft bekeken #821

Closed
opened 2026-07-24 20:26:25 +00:00 by brenno · 5 comments
Owner

Aanleiding

#780 vroeg om de donkere modus als geheel te bekijken. Die ronde is gedaan en
leverde elf bevindingen op, maar hij is niet af: vijf oppervlakken zijn
nooit bekeken. Dat stond in de sluitreactie van #780 en verdwijnt daarmee uit
beeld, dus hier staat het als eigen werk.

Niet nagelopen:

  • Het beamerscherm en het tweede-schermpad (audience_window.dart,
    presenter_displays.dart) — een eigen venster met eigen chrome, en het enige
    oppervlak dat het publiek ziet.
  • De git-panelen en de bladeraars (git/S3/WebDAV) — veel status, veel
    gedempte tekst, en foutmeldingen die je juist in een rotmoment leest.
  • De exportdialoog en de voortgangsmelding.
  • Het kwaliteits- en privacypaneel met de bevindingkaarten en hun
    ernstbanden. Het paneel zelf is in #780 gemeten; de kaarten eronder niet.
  • 24 van de 26 slide-editors. Alleen de cockpit- en titeleditor zijn
    bekeken.

Waarom dit niet "waarschijnlijk goed" is

De grondslag van #780 was dat élke keer dat er gekeken werd, er iets lag. Dat
is in die ronde opnieuw uitgekomen: elf bevindingen, waarvan vier onder 3:1 en
één op 2,08:1. Er is geen reden om aan te nemen dat de vijf oppervlakken die
niemand bekeken heeft de uitzondering zijn.

Twee poorten dekken sinds #780 een deel van de klasse af — de parentabel van de
vast-donkere paletten en de bronwacht op een doorzichtige Colors.white/black
in een TextStyle — maar allebei meten ze tokens en bronregels, niet of een
scherm er goed uitziet. Een oppervlak dat een token op de verkeerde achtergrond
zet, of een asset die niet transparant is, valt er nog steeds buiten.

Wat de ronde moet opleveren

Zelfde vorm als #780: de app draaiend in het profiel Donker, oppervlak voor
oppervlak, en per bevinding een gemeten verhouding in plaats van "ziet er
donker uit". Uitkomst mag "niets gevonden" zijn — dan is de rondgang zelf het
bewijs.

En, net als daar, per bevinding de vraag of er een poort onder kan. Wat via
ThemeData of een palet-token gaat is te meten; wat met de hand geschilderd
wordt pas nadat het onder een palet is gebracht.

Buiten scope

  • #811 (zwevende veldlabels) — staat apart en is onderhanden.
  • Het lichte thema. #780 heeft die vraag beantwoord voor het kwaliteitspaneel
    (daar was licht juist erger); een volledige omgekeerde rondgang is werk voor
    daarna.
## Aanleiding #780 vroeg om de donkere modus als geheel te bekijken. Die ronde is gedaan en leverde elf bevindingen op, maar hij is **niet af**: vijf oppervlakken zijn nooit bekeken. Dat stond in de sluitreactie van #780 en verdwijnt daarmee uit beeld, dus hier staat het als eigen werk. Niet nagelopen: - **Het beamerscherm en het tweede-schermpad** (`audience_window.dart`, `presenter_displays.dart`) — een eigen venster met eigen chrome, en het enige oppervlak dat het publiek ziet. - **De git-panelen en de bladeraars** (git/S3/WebDAV) — veel status, veel gedempte tekst, en foutmeldingen die je juist in een rotmoment leest. - **De exportdialoog en de voortgangsmelding.** - **Het kwaliteits- en privacypaneel met de bevindingkaarten** en hun ernstbanden. Het paneel zelf is in #780 gemeten; de kaarten eronder niet. - **24 van de 26 slide-editors.** Alleen de cockpit- en titeleditor zijn bekeken. ## Waarom dit niet "waarschijnlijk goed" is De grondslag van #780 was dat élke keer dat er gekeken werd, er iets lag. Dat is in die ronde opnieuw uitgekomen: elf bevindingen, waarvan vier onder 3:1 en één op 2,08:1. Er is geen reden om aan te nemen dat de vijf oppervlakken die niemand bekeken heeft de uitzondering zijn. Twee poorten dekken sinds #780 een deel van de klasse af — de parentabel van de vast-donkere paletten en de bronwacht op een doorzichtige `Colors.white/black` in een `TextStyle` — maar allebei meten ze tokens en bronregels, niet of een scherm er goed uitziet. Een oppervlak dat een token op de verkeerde achtergrond zet, of een asset die niet transparant is, valt er nog steeds buiten. ## Wat de ronde moet opleveren Zelfde vorm als #780: de app draaiend in het profiel *Donker*, oppervlak voor oppervlak, en per bevinding een gemeten verhouding in plaats van "ziet er donker uit". Uitkomst mag "niets gevonden" zijn — dan is de rondgang zelf het bewijs. En, net als daar, per bevinding de vraag of er een poort onder kan. Wat via `ThemeData` of een palet-token gaat is te meten; wat met de hand geschilderd wordt pas nadat het onder een palet is gebracht. ## Buiten scope - #811 (zwevende veldlabels) — staat apart en is onderhanden. - Het lichte thema. #780 heeft die vraag beantwoord voor het kwaliteitspaneel (daar was licht juist erger); een volledige omgekeerde rondgang is werk voor daarna.
Author
Owner

Opgepakt. Tak: fix/donkere-modus-ronde-twee-821. Volgorde: beamerscherm en tweede-schermpad eerst (dat is het enige oppervlak dat het publiek ziet), dan de bevindingkaarten, de exportdialoog, de bladeraars, en als laatste de slide-editors op rij.

Opgepakt. Tak: `fix/donkere-modus-ronde-twee-821`. Volgorde: beamerscherm en tweede-schermpad eerst (dat is het enige oppervlak dat het publiek ziet), dan de bevindingkaarten, de exportdialoog, de bladeraars, en als laatste de slide-editors op rij.
Author
Owner

Eerste PR: #823. Dit is niet de hele ronde — het is de statisch vindbare helft.

Wat erin zit (met een gemeten verhouding per bevinding):

  • De foutkant van vijftien ernst-switches die de tokenmigratie van #606 oversloeg — result.isValid ? warningFg : Colors.red.shade700, 2,68 tot 3,97:1. Geraakt: Markdown-editor (foutregel + gutter), oefenrun-samenvatting, exportdialoog + kwaliteitssectie, verwijder-item in het slide-menu, TeX-terugval op een dia, twee bladeraar-iconen, intrekken-knop.
  • Twee vaste witte oppervlakken met mode-afhankelijke tekst erop: het notitieveld (1,1:1) en de opslagkaart (1,7:1).

Twee nieuwe bronwachten: kale Material-kleuren als tekst/icoon, en surface: Colors.white in de notitie-factory. Er zijn nu vier bekende ontsnappingsroutes langs de contrastpoorten; drie ervan hebben een poort, de vierde (vast wit oppervlak) laat zich niet zonder valse positieven vangen en blijft handwerk.

Wat er nog NIET bekeken is — en dit is waarom het issue open blijft:

  • Het beamerscherm en het tweede-schermpad. Dat vraagt twee schermen; niet te doen met één display en computer-use, dus dit is echt onbekeken.
  • De bevindingkaarten met hun ernstbanden (het privacy-/kwaliteitspaneel). Het paneel zelf is in #780 gemeten, de kaarten eronder niet.
  • De meeste slide-editors op rij. Alleen cockpit, titel en de tabel-/Markdown-paden zijn langsgekomen.

Dat is geen 'waarschijnlijk goed'. De statische helft leverde alweer zeventien bevindingen op; de oppervlakken die alleen met eigen ogen te keuren zijn, zijn niet minder verdacht. Ze blijven werk.

Eerste PR: #823. Dit is **niet** de hele ronde — het is de statisch vindbare helft. **Wat erin zit** (met een gemeten verhouding per bevinding): - De foutkant van vijftien ernst-switches die de tokenmigratie van #606 oversloeg — `result.isValid ? warningFg : Colors.red.shade700`, 2,68 tot 3,97:1. Geraakt: Markdown-editor (foutregel + gutter), oefenrun-samenvatting, exportdialoog + kwaliteitssectie, verwijder-item in het slide-menu, TeX-terugval op een dia, twee bladeraar-iconen, intrekken-knop. - Twee vaste witte oppervlakken met mode-afhankelijke tekst erop: het notitieveld (1,1:1) en de opslagkaart (1,7:1). Twee nieuwe bronwachten: kale Material-kleuren als tekst/icoon, en `surface: Colors.white` in de notitie-factory. Er zijn nu vier bekende ontsnappingsroutes langs de contrastpoorten; drie ervan hebben een poort, de vierde (vast wit oppervlak) laat zich niet zonder valse positieven vangen en blijft handwerk. **Wat er nog NIET bekeken is** — en dit is waarom het issue open blijft: - **Het beamerscherm en het tweede-schermpad.** Dat vraagt twee schermen; niet te doen met één display en computer-use, dus dit is echt onbekeken. - **De bevindingkaarten met hun ernstbanden** (het privacy-/kwaliteitspaneel). Het paneel zelf is in #780 gemeten, de kaarten eronder niet. - **De meeste slide-editors op rij.** Alleen cockpit, titel en de tabel-/Markdown-paden zijn langsgekomen. Dat is geen 'waarschijnlijk goed'. De statische helft leverde alweer zeventien bevindingen op; de oppervlakken die alleen met eigen ogen te keuren zijn, zijn niet minder verdacht. Ze blijven werk.
Author
Owner

Verder met de bevindingkaarten en hun ernstbanden. Tak: fix/donkere-modus-bevindingkaarten-821. Reikwijdte: finding_editor, het kwaliteits-/privacypaneel-detail, en de ernstband-chrome — de kaart óp een dia (finding/findingsSummary) blijft vast, want die rendert in een export-isolate.

Verder met de bevindingkaarten en hun ernstbanden. Tak: fix/donkere-modus-bevindingkaarten-821. Reikwijdte: finding_editor, het kwaliteits-/privacypaneel-detail, en de ernstband-chrome — de kaart óp een dia (finding/findingsSummary) blijft vast, want die rendert in een export-isolate.
Author
Owner

Bevindingkaarten gemerged in c06558d9 (PR #824), en op main geverifieerd.

Wat erin zit — met eigen ogen nagekeken in de draaiende bevindingeditor, over alle vijf de banden door de CVSS-vector te variëren:

  • Het scorebadge-label was vast wit op de ernstband. Op de lichte oranje/amber banden (High 3,56:1, Medium 3,19:1) is dat onder AA voor een klein label. Nu via AppTheme.labelOn: zwart op High/Medium, wit op Critical/Low/None. Bevestigd: 'Basis 7.1 · High' en 'Basis 5.1 · Medium' tonen zwart op hun vulling, Critical en Low houden wit.
  • De ernstlegenda in het managementoverzicht: de rode stip zakte op de donkere dialoog naar 2,26:1. Kleur blijft (deelt met de dia), haarlijn erbij zodat de stip als vorm zichtbaar blijft; de ernst staat als tekst ernaast.

Poort: labelOn haalt op élke band de tekstlat (finding_severity_palette_test), één keer rood gezien.

Nog steeds open, en dit issue blijft daarom open:

  • Het beamerscherm en het tweede-schermpad. Vraagt twee schermen; met één display en computer-use niet te keuren. Dit is de enige echt onbekeken hoofdcategorie.
  • De meeste slide-editors op rij. Tot nu langsgekomen: cockpit, titel, tabel/Markdown, en nu de bevindingeditor. De overige ~20 (scorecard, tijdlijn, vraag, checklist, scope-matrix, ondertekening, grafiek, …) niet stuk voor stuk in donkere modus bekeken.

Voor het beamerscherm geldt: dat is een aparte sessie-opzet nodig (twee displays), dus dat blijft liggen tot iemand daar de omgeving voor heeft.

Bevindingkaarten gemerged in `c06558d9` (PR #824), en op main geverifieerd. **Wat erin zit** — met eigen ogen nagekeken in de draaiende bevindingeditor, over alle vijf de banden door de CVSS-vector te variëren: - Het scorebadge-label was vast wit op de ernstband. Op de lichte oranje/amber banden (High 3,56:1, Medium 3,19:1) is dat onder AA voor een klein label. Nu via `AppTheme.labelOn`: zwart op High/Medium, wit op Critical/Low/None. Bevestigd: 'Basis 7.1 · High' en 'Basis 5.1 · Medium' tonen zwart op hun vulling, Critical en Low houden wit. - De ernstlegenda in het managementoverzicht: de rode stip zakte op de donkere dialoog naar 2,26:1. Kleur blijft (deelt met de dia), haarlijn erbij zodat de stip als vorm zichtbaar blijft; de ernst staat als tekst ernaast. Poort: `labelOn` haalt op élke band de tekstlat (`finding_severity_palette_test`), één keer rood gezien. **Nog steeds open, en dit issue blijft daarom open:** - **Het beamerscherm en het tweede-schermpad.** Vraagt twee schermen; met één display en computer-use niet te keuren. Dit is de enige echt onbekeken hoofdcategorie. - **De meeste slide-editors op rij.** Tot nu langsgekomen: cockpit, titel, tabel/Markdown, en nu de bevindingeditor. De overige ~20 (scorecard, tijdlijn, vraag, checklist, scope-matrix, ondertekening, grafiek, …) niet stuk voor stuk in donkere modus bekeken. Voor het beamerscherm geldt: dat is een aparte sessie-opzet nodig (twee displays), dus dat blijft liggen tot iemand daar de omgeving voor heeft.
Author
Owner

Sluiten op verzoek van de beheerder. De rondgang heeft zijn werk gedaan: samen met #780 leverde hij bevindingen in bijna elk bekeken oppervlak, over meerdere PR's — de vier ontsnappingsroutes langs de contrastpoorten, het notitieveld en de opslagkaart, de foutkant van elke ernst-switch, en de scorebadges.

Het resterende werk dat níet met eigen ogen gekeurd is — het beamerscherm/tweede-schermpad (vraagt twee schermen) en de resterende slide-editors per stuk — staat als aparte, smalle kijkronde in #825, zodat het niet in een gesloten thread verdwijnt.

Sluiten op verzoek van de beheerder. De rondgang heeft zijn werk gedaan: samen met #780 leverde hij bevindingen in bijna elk bekeken oppervlak, over meerdere PR's — de vier ontsnappingsroutes langs de contrastpoorten, het notitieveld en de opslagkaart, de foutkant van elke ernst-switch, en de scorebadges. Het resterende werk dat níet met eigen ogen gekeurd is — het beamerscherm/tweede-schermpad (vraagt twee schermen) en de resterende slide-editors per stuk — staat als aparte, smalle kijkronde in #825, zodat het niet in een gesloten thread verdwijnt.
brenno 2026-07-24 22:06:00 +00:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
LibreKAT/Ocideck#821
No description provided.