fix(a11y): de uitleg uit een lege lijst was de slechtst leesbare tekst (#779) #786
No reviewers
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!786
Loading…
Reference in a new issue
No description provided.
Delete branch "fix/palet-contrast-779"
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?
Closes #779.
Correctie op het issue
Het issue noemde 3,31:1 voor de zoekhint. Dat getal klopte niet voor die
hint: ik had tegen
surface2gemeten, en het zoekveld staat opbg. Nagemetentegen de échte ondergrond is de hint 4,12:1.
De bevinding wordt daar niet kleiner van, maar wel scherper:
textDimhaalt alstekst op geen enkel oppervlak van dit palet de 4,5:1 — tussen 3,31 (op
surface2) en 4,24 (opbgDeepest). Het is dus geen randgeval van één veldmaar een token dat structureel de verkeerde lat draagt.
Wat het was
textDimkleurde twee soorten dingen:De zeven tekstplekken zijn de zoekhint, de tagvoortgang, twee beschrijvingshints
en — de vervelendste — de twee lege-toestandregels: "Zet het filter uit om alles
weer te zien" en "Gebruik Bladeren om afbeeldingen van elke locatie te
kiezen". De zin die iemand uit een lege lijst moet helpen, was de slechtst
leesbare tekst in het venster.
Wat het nu is
De zeven tekstgebruiken nemen
textMuted— dat haalt op elk van de zevenoppervlakken ten minste 4,95:1 en blijft ruim onderscheidbaar van
text(10,6).De hint leest dus nog steeds als hint; dat is de eis uit #583.
Wat overblijft heet
iconDim. De naam draagt de regel, want eentextDimdie geen tekst mag kleuren is een val voor de volgende die hem pakt — dezelfde
overweging als
BrandLogoin #735.De oorzaak zat een laag hoger
Beide losstaande donkere paletten — de kiezer én de presentatiemodus — staan
bewust buiten
AppTheme, en dáárom zondert de rauwe-kleurratchet ze uit. Dieuitzondering is verdedigbaar. Wat niemand had gezien is dat ze daarmee ook
buiten elke contrastmeting vielen: ze kwamen in geen enkel testbestand voor.
test/standalone_palette_contrast_test.dartdekt nu beide, in de vorm vanappearance_contrast.dart(#750): per token de rol, per rol de lat, gemetentegen elk oppervlak waar het écht op ligt. Plus de witte labels op de gekleurde
vullingen — dezelfde klasse als het knoplabel dat in #750 op 2,54:1 bleek te
staan, en tot nu toe nergens gemeten.
De presentatiemodus bleek in orde (9,0–15,7:1). Dat is geen reden om hem eruit
te laten: er hield niets dat zo.
Tegenproef
textMutedverdonkerd naar de oude waardesuccessStronglicht gemaakt{successStrong: 1.38}— het witte labelEr staat ook een toets die vastlegt dat
iconDimde tékstlat niet haalt. Dieis er bewust: trekt iemand die waarde ooit op, dan valt hij, en dan is de vraag
of
iconDimentextMutednog twee tinten zijn of één.Wat er níét in zit
De vijf tokens die het issue als "lager dan 4,5" noemde behalve
textDim—accentStrong,dangerStrong,successStrong,danger— zijn nagelopen opgebruik en het zijn vullingen (
backgroundColor:). Die tegen de bodytekst-latleggen is de categoriefout die de helft van #606's basislijn vulde. Wat er bij een
vulling wél toe doet is het label eróp, en dát is nu wel gemeten.
Ook niet: de oppervlakken zelf met eigen ogen bekeken. Dat hoort bij #780.
Poorten
make checkexit 0. Geen l10n (geen nieuwe zichtbare tekst), geenafhankelijkheid, geen SBOM-gevolg.
docs/ACCESSIBILITY.mdenSOURCE_MAP.mdbij.3ff066b09d839cca6dde