Fix declaration-property-value-no-unknown false positives for `attr…

…()` and `if()`
stylelint · jeddy3 · Nov 27, 2025 · Nov 27, 2025 · Nov 27, 2025 · Nov 27, 2025
commit 85307bac830cb3c3fc35f0a8c5d00b4e2df47abf
diff --git a/lib/rules/declaration-property-value-no-unknown/__tests__/index.mjs b/lib/rules/declaration-property-value-no-unknown/__tests__/index.mjs
@@ -89,6 +89,18 @@ testRule({
 		{
 			code: 'a { anchor-name: --bar, --qux; }',
 		},
+		{
+			code: 'a { filter: alpha(opacity=30); }',
+			description: 'ignore csstree parse errors',
+		},
+		{
+			code: 'a { color: if(media(all): red;); }',
+			description: 'ignore csstree parse errors for `if()` function',
+		},
+		{
+			code: 'a { content: attr(data-foo type(<string>)) }',
+			description: 'ignore csstree parse errors for `attr()` function',
+		},
 		{
 			code: '@font-face { font-weight: bolder; }',
 			description: 'ignore descriptors with unknown values',
@@ -344,14 +356,6 @@ testRule({
 			endLine: 1,
 			endColumn: 17,
 		},
-		{
-			code: 'a { filter: alpha(opacity=30); }',
-			message: messages.rejectedParseError('filter', 'alpha(opacity=30)'),
-			line: 1,
-			column: 13,
-			endLine: 1,
-			endColumn: 30,
-		},
 		{
 			code: stripIndent`
 				a { --foo: 10px; }

diff --git a/lib/rules/declaration-property-value-no-unknown/index.cjs b/lib/rules/declaration-property-value-no-unknown/index.cjs
diff --git a/lib/rules/declaration-property-value-no-unknown/index.mjs b/lib/rules/declaration-property-value-no-unknown/index.mjs
@@ -22,8 +22,6 @@
 
 const messages = ruleMessages(ruleName, {
 	rejected: (property, value) => `Unexpected unknown value "${value}" for property "${property}"`,
-	rejectedParseError: (property, value) =>
-		`Cannot parse property value "${value}" for property "${property}"`,
 });
 
 const meta = {
@@ -156,19 +154,7 @@
 
 				if (containsUnsupportedFunction(cssTreeValueNode)) return;
 			} catch {
-				const index = declarationValueIndex(decl);
-				const endIndex = index + value.length;
-
-				report({
-					message: messages.rejectedParseError,
-					messageArgs: [prop, value],
-					node: decl,
-					index,
-					endIndex,
-					result,
-					ruleName,
-				});
-
+				// Ignore parse errors
 				return;
 			}
 
@@ -250,6 +236,6 @@
 }

 rule.ruleName = ruleName;
 rule.messages = messages;
 rule.meta = meta;
 export default rule;