diff options
| author | Craig Jennings <c@cjennings.net> | 2026-06-16 06:11:15 -0500 |
|---|---|---|
| committer | Craig Jennings <c@cjennings.net> | 2026-06-16 06:11:15 -0500 |
| commit | 9e99749de911ffc5b375bc79ee664d498e4d76d6 (patch) | |
| tree | f76652c5d684536598f8d510baf52f304c70d818 /scripts/theme-studio/app-util.js | |
| parent | cf882dfe168463471598d01205256131bc4e7f1b (diff) | |
| download | dotemacs-9e99749de911ffc5b375bc79ee664d498e4d76d6.tar.gz dotemacs-9e99749de911ffc5b375bc79ee664d498e4d76d6.zip | |
feat(theme-studio): move the contrast verdict into a hover
The contrast column showed "5.4 PASS". The number's color already encodes the tier (green AAA, grey AA, red fail), so the PASS/FAIL word was redundant. I dropped it and put the WCAG meaning in the cell's hover via a pure contrastTitle helper. crHtml now renders just the colored number. verdictFor stays for the covered-overlay worst-case readout, which is unchanged.
Diffstat (limited to 'scripts/theme-studio/app-util.js')
| -rw-r--r-- | scripts/theme-studio/app-util.js | 12 |
1 files changed, 11 insertions, 1 deletions
diff --git a/scripts/theme-studio/app-util.js b/scripts/theme-studio/app-util.js index e3f76dd88..774553012 100644 --- a/scripts/theme-studio/app-util.js +++ b/scripts/theme-studio/app-util.js @@ -17,4 +17,14 @@ function ratingColor(r){return r>=7?'#5d9b86':r>=4.5?'#a9b2bb':'#cb6b4d';} // Pick black or white text for a background hex, by WCAG relative luminance. function textOn(h){const L=rl(h);return ((L+0.05)/0.05)>(1.05/(L+0.05))?'#000':'#fff';} -export { normHex, ratingColor, textOn }; +// Hover text for a contrast ratio. The number's color already encodes the tier +// (ratingColor: green AAA, grey AA, red fail), so the cell drops the PASS/FAIL +// word and this explains the color on hover. +function contrastTitle(r){ + const n=r.toFixed(1)+':1'; + if(r>=7) return n+' (green): passes WCAG AA and AAA'; + if(r>=4.5) return n+' (grey): passes WCAG AA, not AAA'; + return n+' (red): fails WCAG AA'; +} + +export { normHex, ratingColor, textOn, contrastTitle }; |
