Make non-absolute colors invalid in @font-palette-values/override-colors

cdoublev · cdoublev · commit cbb8c23b9d7b · 2025-04-25T14:29:35.000+02:00
diff --git a/__tests__/stylesheet.js b/__tests__/stylesheet.js
@@ -1468,6 +1468,11 @@ describe('CSS grammar - semantic', () => {
 
                 style {}
                 color: red;
+                override-colors: 0 currentcolor;
+                override-colors: 0 AccentColor;
+                override-colors: 0 light-dark(red, red);
+                override-colors: 0 contrast-color(red);
+                override-colors: 0 device-cmyk(0 0 0 0);
 
                 base-palette: initial;
                 base-palette: inherit(--custom);
diff --git a/lib/parse/postprocess.js b/lib/parse/postprocess.js
@@ -250,6 +250,25 @@ function postParseCalcValue(value) {
     return value.types[0] === '<simple-block>' ? value.value : value
 }
 
+/**
+ * @param {object} color
+ * @param {object} node
+ * @returns {SyntaxError|object}
+ * @see {@link https://drafts.csswg.org/css-fonts-4/#descdef-font-palette-values-override-colors}
+ *
+ * It aborts parsing when the color is not an absolute color for override-colors
+ * in @font-palette-values.
+ */
+function postParseColor(color, node) {
+    if (
+        node.context.declaration?.definition.name === 'override-colors'
+        && !color.types.includes('<color-base>')
+    ) {
+        return error(node)
+    }
+    return color
+}
+
 /**
  * @param {object|object[]} schemes
  * @param {object} node
