better handling of function-no-unknown

apply pr comments
stylelint · Feb 15, 2022 · a58f929 · a58f929
1 parent 42a401c
commit a58f929
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 0 deletions.
diff --git a/lib/rules/function-no-unknown/__tests__/index.js b/lib/rules/function-no-unknown/__tests__/index.js
@@ -54,6 +54,9 @@ testRule({
 		{
 			code: 'a { $list: (list) }',
 		},
+		{
+			code: 'a { --primary-color: #{theme(1px)}; }',
+		},
 	],
 });
 

diff --git a/lib/utils/__tests__/isStandardSyntaxFunction.test.js b/lib/utils/__tests__/isStandardSyntaxFunction.test.js
@@ -20,6 +20,12 @@ describe('isStandardSyntaxFunction', () => {
 	it('scss map', () => {
 		expect(isStandardSyntaxFunction(func('a { $map: (key: value) }'))).toBe(false);
 	});
+
+	it('scss function in custom prop', () => {
+		expect(isStandardSyntaxFunction(func('a { --primary-color: #{darken(#fff, 0.2)} }'))).toBe(
+			false,
+		);
+	});
 });
 
 function func(css) {

diff --git a/lib/utils/isStandardSyntaxFunction.js b/lib/utils/isStandardSyntaxFunction.js
@@ -12,5 +12,9 @@ module.exports = function (node) {
 		return false;
 	}
 
+	if (node.value.startsWith('#{')) {
+		return false;
+	}
+
 	return true;
 };