Stijl: het Vigilis-accentgeel is onleesbaar als linkkleur op wit #1819
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#1819
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?
Wat er staat
Het gebundelde Vigilis-profiel heeft
accentColor: '#FFB800'opslideBackgroundColor: '#FFFFFF'. Contrastverhouding 1,73.Het accent is niet alleen decoratie: de previews geven het door als
linkColor(
bullets_previews.dart,table_preview.dart). Een link in een Vigilis-deck isdus geel op wit, en dat is als tekst onleesbaar —
SlideQualityAnalyzertoetsthet terecht tegen 4,5 en het haalt zelfs de 3,0 voor grafische elementen niet.
Waarom dit een apart issue is
Dit is een merkbesluit, geen technische keuze:
#FFB800is de merkkleur vanVigilis. Om als tekst 4,5 te halen moet het naar ongeveer
#96690F(4,86), endat verandert het zichtbare accent overal — bulletmarkers, links, tabelkop,
randen. Dat hoort de eigenaar van het merk te besluiten, niet een reparatiebeurt.
Bij #1818 is daarom bewust gekozen: de sectiedia is wél gerepareerd (die tekende
wit op ditzelfde geel, 1,73 — dat is geen merkbesluit maar onleesbaar), en de
accentkleur blijft staan. De test die daar landt eist nul contrastbevindingen
voor de gebundelde profielen, mét dit ene geval als benoemde uitzondering die
naar dit issuenummer verwijst — zodat een níeuwe kleurfout alsnog rood geeft en
deze niet stilzwijgend meelift.
Mogelijke uitkomsten
#96690F) — haalt 4,86, merk verandert zichtbaar.accentColor, zodat het geelalleen nog grafisch wordt gebruikt. Dan geldt 3,0 in plaats van 4,5 — maar
1,73 haalt ook die niet, dus dit lost op zichzelf niets op.
Wordt het 1 of 2, dan gaat de uitzondering uit de test in
test/theme_profile_contrast_test.dart. Wordt het 3, dan verhuist de redendaarheen zodat hij niet in een issue blijft hangen.
Besloten: het merkgeel #FFB800 blijft staan als bewuste keuze. De reden is verhuisst van dit issue naar de test zelf (
test/theme_profile_contrast_test.dart), zodat hij niet in een issue blijft hangen. De uitzondering in_aanvaardis nu zelfstandig — niet meer afhankelijk van een issuenummer. Opgelost in PR #1823 (gemerged).