De donkere modus, ronde twee: de vijf oppervlakken die #780 niet heeft bekeken #821
Labels
No labels
accepted
bug
declined
docs
duplicate
enhancement
good first issue
in-progress
needs-info
privacy
security
triage
No milestone
No project
No assignees
1 participant
Notifications
Due date
No due date set.
Dependencies
No dependencies set
Reference
LibreKAT/Ocideck#821
Loading…
Reference in a new issue
No description provided.
Delete branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
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:
audience_window.dart,presenter_displays.dart) — een eigen venster met eigen chrome, en het enigeoppervlak dat het publiek ziet.
gedempte tekst, en foutmeldingen die je juist in een rotmoment leest.
ernstbanden. Het paneel zelf is in #780 gemeten; de kaarten eronder niet.
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/blackin een
TextStyle— maar allebei meten ze tokens en bronregels, niet of eenscherm 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
ThemeDataof een palet-token gaat is te meten; wat met de hand geschilderdwordt pas nadat het onder een palet is gebracht.
Buiten scope
(daar was licht juist erger); een volledige omgekeerde rondgang is werk voor
daarna.
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.Eerste PR: #823. Dit is niet de hele ronde — het is de statisch vindbare helft.
Wat erin zit (met een gemeten verhouding per bevinding):
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 nieuwe bronwachten: kale Material-kleuren als tekst/icoon, en
surface: Colors.whitein 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:
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.
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.
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:
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.Poort:
labelOnhaalt op élke band de tekstlat (finding_severity_palette_test), één keer rood gezien.Nog steeds open, en dit issue blijft daarom open:
Voor het beamerscherm geldt: dat is een aparte sessie-opzet nodig (twee displays), dus dat blijft liggen tot iemand daar de omgeving voor heeft.
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.