fix(a11y): de bevindingkaarten — het scorebadge-label was wit op de lichte ernstbanden (#821) #824

Merged

brenno merged 2 commits from fix/donkere-modus-bevindingkaarten-821 into main 2026-07-24 22:02:20 +00:00

Conversation 0 Commits 2 Files changed 6 +49 -7

brenno commented 2026-07-24 21:58:27 +00:00

Owner

Copy link

Verder met de bevindingkaarten uit #821 — een van de oppervlakken die #780
niet had bekeken.

Wat er mis was

Twee dingen, allebei op de FIRST-ernstbanden.

Het gevulde scorebadge. De bevindingeditor en de CVSS-bouwer zetten een
vast wit label (12px, bold) op de ernstband. Dat is gewone tekst, lat 4,5:1 —
en op de lichte oranje/amber banden haalde wit dat niet:

band	wit label
High (oranje #EA580C)	3,56 : 1
Medium (amber #D97706)	3,19 : 1

Op een dia is hetzelfde label groot (~30px) en volstaat 3:1; daar is wit goed.
In de editor is het klein.

De ernstlegenda in het managementoverzicht. Een gekleurde stip per band,
en op de donkere dialoog zakt de rode naar 2,26:1 — die verdwijnt bijna.

De fix

AppTheme.labelOn (nu publiek, was _labelOn) kiest zwart of wit per band:
zwart op High/Medium (5,90 / 6,59:1), wit op Critical/Low/None. De badges in
beide plekken lezen hun label nu zo.

De legenda-stip houdt zijn FIRST-kleur (die deelt hij met de dia) maar krijgt
een haarlijn, zodat hij als vórm zichtbaar blijft op elke achtergrond. De ernst
staat als tekst ernaast — de kleur is bevestiging, niet de enige drager (WCAG
1.4.1).

Draaiend nagekeken

De bevindingeditor in donkere modus, over alle vijf de banden door de
CVSS-vector te variëren:

• Basis 7.1 · High → zwart label op oranje ✓
• Basis 5.1 · Medium → zwart label op amber ✓
• Critical (9.3) → wit op rood ✓
• Low (2.1) → wit op groen ✓

Precies de gevallen die de fix raakt, met eigen ogen bevestigd.

Poort

finding_severity_palette_test: labelOn haalt op élke band de tekstlat.
Eén keer rood gezien (medium 3,19:1 met vast wit).

make check, goldens, secrets en SAST groen. Bewaker overgeslagen, expliciet:
geen formaat/opslag/afhankelijkheid/verkeer/belofte — kleurtokens en één toets.

Nog open

De rondgang van #821 is hiermee nog niet af. Het beamerscherm en het
tweede-schermpad blijven onbekeken (dat vraagt twee schermen), net als de
meeste slide-editors op rij. #821 blijft open.

Refs #821

Verder met de bevindingkaarten uit #821 — een van de oppervlakken die #780 niet had bekeken. ## Wat er mis was Twee dingen, allebei op de FIRST-ernstbanden. **Het gevulde scorebadge.** De bevindingeditor en de CVSS-bouwer zetten een vast wit label (12px, bold) op de ernstband. Dat is gewone tekst, lat 4,5:1 — en op de lichte oranje/amber banden haalde wit dat niet: | band | wit label | |---|---| | High (oranje #EA580C) | 3,56 : 1 | | Medium (amber #D97706) | 3,19 : 1 | Op een dia is hetzelfde label groot (~30px) en volstaat 3:1; daar is wit goed. In de editor is het klein. **De ernstlegenda in het managementoverzicht.** Een gekleurde stip per band, en op de donkere dialoog zakt de rode naar 2,26:1 — die verdwijnt bijna. ## De fix `AppTheme.labelOn` (nu publiek, was `_labelOn`) kiest zwart of wit per band: zwart op High/Medium (5,90 / 6,59:1), wit op Critical/Low/None. De badges in beide plekken lezen hun label nu zo. De legenda-stip houdt zijn FIRST-kleur (die deelt hij met de dia) maar krijgt een haarlijn, zodat hij als vórm zichtbaar blijft op elke achtergrond. De ernst staat als tekst ernaast — de kleur is bevestiging, niet de enige drager (WCAG 1.4.1). ## Draaiend nagekeken De bevindingeditor in donkere modus, over alle vijf de banden door de CVSS-vector te variëren: - **Basis 7.1 · High** → zwart label op oranje ✓ - **Basis 5.1 · Medium** → zwart label op amber ✓ - Critical (9.3) → wit op rood ✓ - Low (2.1) → wit op groen ✓ Precies de gevallen die de fix raakt, met eigen ogen bevestigd. ## Poort `finding_severity_palette_test`: `labelOn` haalt op élke band de tekstlat. Eén keer rood gezien (medium 3,19:1 met vast wit). `make check`, goldens, secrets en SAST groen. Bewaker overgeslagen, expliciet: geen formaat/opslag/afhankelijkheid/verkeer/belofte — kleurtokens en één toets. ## Nog open De rondgang van #821 is hiermee nog niet af. Het beamerscherm en het tweede-schermpad blijven onbekeken (dat vraagt twee schermen), net als de meeste slide-editors op rij. #821 blijft open. Refs #821

brenno added 2 commits 2026-07-24 21:58:28 +00:00

fix(a11y): het scorebadge-label was wit op de lichte ernstbanden (#821) ... 92b1557a94

Verder in de bevindingkaarten. De gevulde scorebadge in de bevindingeditor en
de CVSS-bouwer zette een vast wit label (12px, bold) op de ernstband. Dat is
gewone tekst — lat 4,5:1 — en op de lichte oranje/amber banden haalde wit dat
niet:

  High  (oranje #EA580C)   3,56 : 1
  Medium (amber #D97706)   3,19 : 1

Op een dia is hetzelfde label groot (~30px) en volstaat 3:1; daar blijft het
wit. In de editor is het klein. `AppTheme.labelOn` (nu publiek, was `_labelOn`)
kiest zwart of wit per band: zwart op High/Medium (5,90 / 6,59:1), wit op
Critical/Low/None (6,47 / 5,02 / 7,58:1).

Draaiend nagekeken over alle vijf de banden: "Basis 7.1 · High" en "Basis 5.1 ·
Medium" tonen nu een zwart label op hun lichte vulling, Critical en Low houden
wit.

En de ernstlegenda in het managementoverzicht: de gekleurde stip is een vaste
FIRST-band, en op de donkere dialoog zakt de rode naar 2,26:1 en verdwijnt
bijna. De kleuren blijven (ze delen hun waarde met de dia); een haarlijn houdt
de stip als vórm zichtbaar. De ernst zelf staat als tekst ernaast, dus de kleur
is bevestiging, niet de enige drager (WCAG 1.4.1).

Bewaakt door een toets in `finding_severity_palette_test`: `labelOn` haalt op
élke band de tekstlat. Eén keer rood gezien door het label weer op vast wit te
zetten (medium 3,19:1).

Refs #821

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

docs(source-map): het scorebadge-label leest de bandkleur via labelOn (#821) ... 4a76dfadee

Refs #821

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

brenno merged commit c06558d959 into main 2026-07-24 22:02:20 +00:00

brenno referenced this pull request from a commit 2026-07-24 22:02:21 +00:00

Merge pull request 'fix(a11y): de bevindingkaarten — het scorebadge-label was wit op de lichte ernstbanden (#821)' (#824) from fix/donkere-modus-bevindingkaarten-821 into main

brenno referenced this pull request 2026-07-24 22:02:57 +00:00

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

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

accepted

Assessed and will be built

  

bug

Something is broken

  

declined

Assessed and will not be built

  

docs

Documentation only

  

duplicate

Already tracked elsewhere

  

enhancement

New feature or improvement request

  

good first issue

Small, self-contained, and a good place to start

  

in-progress

Someone is working on this right now. Do not start it in parallel.

  

needs-info

Waiting for the reporter

  

privacy

Personal data can leak. Handled before anything else.

  

security

A weakness someone could use. Handled before anything else.

  

triage

Awaiting assessment

No labels

accepted

bug

declined

docs

duplicate

enhancement

good first issue

in-progress

needs-info

privacy

security

triage

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

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!824

No description provided.