fix for color contrast

primer · Dec 21, 2023 · 189338f · 189338f
1 parent 0d07455
commit 189338f
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 8 deletions.
diff --git a/scripts/color-contrast.config.ts b/scripts/color-contrast.config.ts
@@ -19,9 +19,8 @@ const baseRequirements: ContrastRequirement[] = [
   [4.5, 'fg.default', 'canvas.inset'],
   [4.5, 'fg.muted', 'canvas.inset'],
   // default text vs link
-  // disbaled because we now have the underline for links
-  // [3, 'fg.default', 'accent.fg'],
-  // [3, 'fg.default', 'fg.muted'],
+  [3, 'fg.default', 'accent.fg'],
+  [3, 'fg.default', 'fg.muted'],
   // default text on role bg
   // TODO: contrast does not work with semi-transparent colors
   [4.5, 'fg.default', 'accent.subtle'],
@@ -109,9 +108,8 @@ const highContrast: ContrastRequirement[] = [
   [7, 'fg.default', 'canvas.inset'],
   [7, 'fg.muted', 'canvas.inset'],
   // default text vs link
-  // disbaled because we now have the underline for links
-  // [4.5, 'fg.default', 'accent.fg'],
-  // [4.5, 'fg.default', 'fg.muted'],
+  [4.5, 'fg.default', 'accent.fg'],
+  [4.5, 'fg.default', 'fg.muted'],
   // default text on role bg
   // TODO: contrast does not work with semi-transparent colors
   [7, 'fg.default', 'accent.subtle'],

diff --git a/scripts/color-contrast.ts b/scripts/color-contrast.ts
@@ -248,9 +248,9 @@ export const check = async (themes?: Themes[], output: 'log' | 'file' | 'all' |
   const results = await checkContrastForThemes(contrastRequirements, themes)
 
   if (output === 'log' || output === 'all') {
-    for (const {resultTable, failingContrast, theme} of results) {
+    for (const {resultTable, failingContrast} of results) {
       // eslint-disable-next-line no-console
-      console.log('\n', '\x1b[1m', `Theme: ${theme}\n`, '\x1b[0m', resultTable, '\n')
+      console.log('\n', resultTable, '\n')
       if (failingContrast > 0) {
         // eslint-disable-next-line no-console
         console.error('❌ Failing contrast checks:', failingContrast, '\n')