Fix selector-no-vendor-prefix end positions (#6261)

stylelint · Aug 14, 2022 · d324493 · d324493
1 parent 43ecd9f
commit d324493
Show file tree

Hide file tree

Showing 2 changed files with 35 additions and 15 deletions.
diff --git a/lib/rules/selector-no-vendor-prefix/__tests__/index.js b/lib/rules/selector-no-vendor-prefix/__tests__/index.js
@@ -51,46 +51,62 @@ testRule({
 			message: messages.rejected(':-webkit-full-screen'),
 			line: 1,
 			column: 1,
+			endLine: 1,
+			endColumn: 21,
 		},
 		{
 			code: ':-wEbKiT-fUlL-sCrEeN a {}',
 			fixed: ':fUlL-sCrEeN a {}',
 			message: messages.rejected(':-wEbKiT-fUlL-sCrEeN'),
 			line: 1,
 			column: 1,
+			endLine: 1,
+			endColumn: 21,
 		},
 		{
 			code: ':-WEBKIT-FULL-SCREEN a {}',
 			fixed: ':FULL-SCREEN a {}',
 			message: messages.rejected(':-WEBKIT-FULL-SCREEN'),
 			line: 1,
 			column: 1,
+			endLine: 1,
+			endColumn: 21,
 		},
 		{
 			code: 'body, :-ms-fullscreen a {}',
 			fixed: 'body, :fullscreen a {}',
 			message: messages.rejected(':-ms-fullscreen'),
 			line: 1,
 			column: 7,
+			endLine: 1,
+			endColumn: 22,
 		},
 		{
 			code: 'input::-moz-placeholder, input::placeholder { color: pink; }',
 			fixed: 'input::placeholder, input::placeholder { color: pink; }',
 			message: messages.rejected('::-moz-placeholder'),
 			line: 1,
 			column: 6,
+			endLine: 1,
+			endColumn: 24,
 		},
 		{
 			code: 'input::-moz-placeholder { color: pink; }',
 			fixed: 'input::placeholder { color: pink; }',
 			message: messages.rejected('::-moz-placeholder'),
 			line: 1,
 			column: 6,
+			endLine: 1,
+			endColumn: 24,
 		},
 		{
 			code: 'input::-webkit-input-placeholder { color: pink; }',
 			fixed: 'input::input-placeholder { color: pink; }',
 			message: messages.rejected('::-webkit-input-placeholder'),
+			line: 1,
+			column: 6,
+			endLine: 1,
+			endColumn: 33,
 		},
 	],
 });

diff --git a/lib/rules/selector-no-vendor-prefix/index.js b/lib/rules/selector-no-vendor-prefix/index.js
@@ -49,25 +49,29 @@ const rule = (primary, secondaryOptions, context) => {
 
 			parseSelector(selector, result, ruleNode, (selectorTree) => {
 				selectorTree.walkPseudos((pseudoNode) => {
-					if (isAutoprefixable.selector(pseudoNode.value)) {
-						if (optionsMatches(secondaryOptions, 'ignoreSelectors', pseudoNode.value)) {
-							return;
-						}
+					const { value } = pseudoNode;
 
-						if (context.fix) {
-							ruleNode.selector = isAutoprefixable.unprefix(ruleNode.selector);
+					if (!isAutoprefixable.selector(value)) {
+						return;
+					}
+
+					if (optionsMatches(secondaryOptions, 'ignoreSelectors', value)) {
+						return;
+					}
 
-							return;
-						}
+					if (context.fix) {
+						ruleNode.selector = isAutoprefixable.unprefix(selector);
 
-						report({
-							result,
-							ruleName,
-							message: messages.rejected(pseudoNode.value),
-							node: ruleNode,
-							index: (ruleNode.raws.before || '').length + pseudoNode.sourceIndex,
-						});
+						return;
 					}
+
+					report({
+						result,
+						ruleName,
+						message: messages.rejected(value),
+						node: ruleNode,
+						word: value,
+					});
 				});
 			});
 		});