Process multiple spaces in media-feature-parentheses-space-inside (#4513

) * Process multiple spaces in media (#4509) * Consider tabs as spaces
stylelint · Jan 9, 2020 · d9dbce2 · d9dbce2
1 parent 1dc203e
commit d9dbce2
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 2 deletions.
diff --git a/lib/rules/media-feature-parentheses-space-inside/__tests__/index.js b/lib/rules/media-feature-parentheses-space-inside/__tests__/index.js
@@ -135,6 +135,34 @@ testRule(rule, {
 	],
 
 	reject: [
+		{
+			code: '@media (  min-width: 700px) {}',
+			fixed: '@media (min-width: 700px) {}',
+			message: messages.rejectedOpening,
+			line: 1,
+			column: 9,
+		},
+		{
+			code: '@media (min-width: 700px  ) {}',
+			fixed: '@media (min-width: 700px) {}',
+			message: messages.rejectedClosing,
+			line: 1,
+			column: 26,
+		},
+		{
+			code: '@media (\t  min-width: 700px) {}',
+			fixed: '@media (min-width: 700px) {}',
+			message: messages.rejectedOpening,
+			line: 1,
+			column: 9,
+		},
+		{
+			code: '@media (min-width: 700px\t) {}',
+			fixed: '@media (min-width: 700px) {}',
+			message: messages.rejectedClosing,
+			line: 1,
+			column: 25,
+		},
 		{
 			code: '@media (max-width: 300px ) {}',
 			fixed: '@media (max-width: 300px) {}',

diff --git a/lib/rules/media-feature-parentheses-space-inside/index.js b/lib/rules/media-feature-parentheses-space-inside/index.js
@@ -39,7 +39,7 @@ function rule(expectation, options, context) {
 					const len = valueParser.stringify(node).length;
 
 					if (expectation === 'never') {
-						if (node.before === ' ') {
+						if (/[ \t]/.test(node.before)) {
 							if (context.fix) node.before = '';
 
 							violations.push({
@@ -48,7 +48,7 @@ function rule(expectation, options, context) {
 							});
 						}
 
-						if (node.after === ' ') {
+						if (/[ \t]/.test(node.after)) {
 							if (context.fix) node.after = '';
 
 							violations.push({