ignore urange productions in unicode-range description

stylelint · Jun 5, 2023 · 184fbd5 · 184fbd5
1 parent 060a4d3
commit 184fbd5
Show file tree

Hide file tree

Showing 2 changed files with 25 additions and 1 deletion.
diff --git a/lib/rules/unit-disallowed-list/__tests__/index.js b/lib/rules/unit-disallowed-list/__tests__/index.js
@@ -5,7 +5,7 @@ const { messages, ruleName } = require('..');
 testRule({
 	ruleName,
 
-	config: ['px', 'vmin'],
+	config: ['px', 'vmin', 'f'],
 
 	accept: [
 		{
@@ -118,6 +118,9 @@ testRule({
 			code: '@media (min-width: 10em)\n  and (max-width: 20em) {}',
 			description: 'complex @media',
 		},
+		{
+			code: '@font-face { unicode-range: U+0100-024F; }',
+		},
 	],
 
 	reject: [
@@ -260,6 +263,24 @@ testRule({
 			endLine: 1,
 			endColumn: 24,
 		},
+		{
+			code: '@font-face { color: U+0100-024F; }',
+			message: messages.rejected('F'),
+			description: 'Unicode range value in something other than `unicode-range`',
+			line: 1,
+			column: 31,
+			endLine: 1,
+			endColumn: 32,
+		},
+		{
+			code: 'a { unicode-range: U+0100-024F; }',
+			message: messages.rejected('F'),
+			description: 'Unicode range value in something other than `@font-face`',
+			line: 1,
+			column: 30,
+			endLine: 1,
+			endColumn: 31,
+		},
 	],
 });
 

diff --git a/lib/rules/unit-disallowed-list/index.js b/lib/rules/unit-disallowed-list/index.js
@@ -20,6 +20,7 @@ const ruleMessages = require('../../utils/ruleMessages');
 const validateObjectWithArrayProps = require('../../utils/validateObjectWithArrayProps');
 const validateOptions = require('../../utils/validateOptions');
 const { isRegExp, isString } = require('../../utils/validateTypes');
+const isUnicodeRangeDescriptor = require('../../utils/isUnicodeRangeDescriptor');
 
 const ruleName = 'unit-disallowed-list';
 
@@ -146,6 +147,8 @@ const rule = (primary, secondaryOptions) => {
 			});
 		});
 		root.walkDecls((decl) => {
+			if (isUnicodeRangeDescriptor(decl)) return;
+
 			const value = getDeclarationValue(decl);
 
 			if (!hasDimension(value)) return;