Contrastpoort mist de ondertitel van titel- en tussentiteldia's #1290

New issue

Closed

opened 2026-08-05 23:34:14 +00:00 by brenno · 2 comments

brenno commented

2026-08-05 23:34:14 +00:00

Owner

Symptoom

Een dia met een donkerblauwe achtergrond en een lichterblauwe titel toont eronder een ondertitel die voor mensen niet te lezen is, zonder dat de kwaliteitscontrole waarschuwt.

Oorzaak

_checkSlideContrast (lib/services/slide_quality_analyzer.dart) toetste de vólle titeltekst tegen de titel- respectievelijk tussentitelachtergrond, maar de ondertitel eronder rendert op verlaagde dekking (kTitleSubtitleAlpha = 0,72 in lib/utils/title_contrast.dart) — een lichtere, en dus zwakker contrasterende, variant van dezelfde kleur. Haalt de volle titel nog net de drempel voor grote tekst, dan zakt de lichtere ondertitel er ruim onder en tóch kwam er geen melding. De over-afbeelding-route (evaluateTitleContrast) nam de ondertitel wél mee; de effen-kleur-route niet.

Oplossing

_checkSlideContrast toetst nu ook de ondertitel (gemengd op de gedeelde constante kTitleSubtitleAlpha, kleur volgt de render inclusief een per-dia titleTextColorOverride) voor zowel de titel- als de tussentiteldia. De 0,72 is nu één constante die render en toets delen, zodat ze niet uit elkaar lopen. Label hergebruikt de bestaande, in 31 talen vertaalde string 'Ondertitel' — geen nieuwe l10n. Regressietest eerst-rood: test/subtitle_contrast_test.dart.

Gevonden tijdens de beeldkeuring van #1279.

## Symptoom Een dia met een donkerblauwe achtergrond en een lichterblauwe titel toont eronder een ondertitel die voor mensen niet te lezen is, zonder dat de kwaliteitscontrole waarschuwt. ## Oorzaak `_checkSlideContrast` (lib/services/slide_quality_analyzer.dart) toetste de vólle titeltekst tegen de titel- respectievelijk tussentitelachtergrond, maar de ondertitel eronder rendert op verlaagde dekking (`kTitleSubtitleAlpha` = 0,72 in lib/utils/title_contrast.dart) — een lichtere, en dus zwakker contrasterende, variant van dezelfde kleur. Haalt de volle titel nog net de drempel voor grote tekst, dan zakt de lichtere ondertitel er ruim onder en tóch kwam er geen melding. De over-afbeelding-route (`evaluateTitleContrast`) nam de ondertitel wél mee; de effen-kleur-route niet. ## Oplossing `_checkSlideContrast` toetst nu ook de ondertitel (gemengd op de gedeelde constante `kTitleSubtitleAlpha`, kleur volgt de render inclusief een per-dia `titleTextColorOverride`) voor zowel de titel- als de tussentiteldia. De 0,72 is nu één constante die render en toets delen, zodat ze niet uit elkaar lopen. Label hergebruikt de bestaande, in 31 talen vertaalde string 'Ondertitel' — geen nieuwe l10n. Regressietest eerst-rood: test/subtitle_contrast_test.dart. Gevonden tijdens de beeldkeuring van #1279.

brenno added the

bug

in-progress

labels

2026-08-05 23:34:14 +00:00

brenno commented

2026-08-05 23:34:36 +00:00

Author

Owner

Opgepakt en gebouwd op tak fix/1279-split-fit-scale (commit 1552801c), samen met #1279 waar dit tijdens de keuring uitkwam. Eerste-rood regressietest test/subtitle_contrast_test.dart. Sluit ik na groene make check + merge op main.

Opgepakt en gebouwd op tak fix/1279-split-fit-scale (commit 1552801c), samen met #1279 waar dit tijdens de keuring uitkwam. Eerste-rood regressietest test/subtitle_contrast_test.dart. Sluit ik na groene make check + merge op main.

brenno referenced this issue from a pull request that will close it,

2026-08-05 23:55:57 +00:00

fix(split+quality): paginadoel volgt de tekstkolom naast een afbeelding (#1279) + ondertitel-contrasttoets (#1290) #1292

brenno closed this issue

2026-08-06 00:00:31 +00:00

brenno referenced this issue from a commit

2026-08-06 00:00:32 +00:00

Merge pull request 'fix(split+quality): paginadoel volgt de tekstkolom naast een afbeelding (#1279) + ondertitel-contrasttoets (#1290)' (#1292) from fix/1279-split-fit-scale into main

brenno removed the

in-progress

label

2026-08-06 00:01:10 +00:00

brenno commented

2026-08-06 00:01:14 +00:00

Author

Owner

Gemerged op main via PR #1292 (merge-commit abcfab80). _checkSlideContrast toetst nu ook de 0,72-dekking-ondertitel van titel- en tussentiteldia's, kleur volgt de render inclusief per-dia override. Eerst-rood regressietest test/subtitle_contrast_test.dart. Geen nieuwe l10n (label 'Ondertitel' hergebruikt).

Gemerged op main via PR #1292 (merge-commit abcfab80). _checkSlideContrast toetst nu ook de 0,72-dekking-ondertitel van titel- en tussentiteldia's, kleur volgt de render inclusief per-dia override. Eerst-rood regressietest test/subtitle_contrast_test.dart. Geen nieuwe l10n (label 'Ondertitel' hergebruikt).

brenno referenced this issue

2026-08-20 16:38:20 +00:00

De kop en de kop-/voetband van een document worden op contrast getoetst #1616