Fix selector-not-notation autofix of the "simple" option (#7703)

stylelint · May 23, 2024 · 8364760 · 8364760
1 parent c4ea9d4
commit 8364760
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 2 deletions.
diff --git a/.changeset/eight-falcons-scream.md b/.changeset/eight-falcons-scream.md
@@ -0,0 +1,5 @@
+---
+"stylelint": patch
+---
+
+Fixed: `selector-not-notation` autofix of the `"simple"` option
diff --git a/lib/rules/selector-not-notation/__tests__/index.mjs b/lib/rules/selector-not-notation/__tests__/index.mjs
@@ -67,6 +67,15 @@ testRule({
  endLine: 1,
  endColumn: 13,
  },
+ {
+ code: 'p, :not(a, div) {}',
+ fixed: 'p, :not(a):not(div) {}',
+ message: messages.expected('simple'),
+ line: 1,
+ column: 4,
+ endLine: 1,
+ endColumn: 16,
+ },
  {
  code: ':not(.bar, .baz, .foo) {}',
  fixed: ':not(.bar):not(.baz):not(.foo) {}',

diff --git a/lib/rules/selector-not-notation/index.cjs b/lib/rules/selector-not-notation/index.cjs
@@ -166,8 +166,10 @@ function fixSimple(not) {
 
  for (const s of simpleSelectors) {
  const last = getLastConsecutiveNot(not);
+ const clone = last.clone({ nodes: [s] });
 
- not.parent.insertAfter(last, last.clone({ nodes: [s] }));
+ clone.rawSpaceBefore = '';
+ not.parent.insertAfter(last, clone);
  }
 }
 

diff --git a/lib/rules/selector-not-notation/index.mjs b/lib/rules/selector-not-notation/index.mjs
@@ -162,8 +162,10 @@ function fixSimple(not) {
 
  for (const s of simpleSelectors) {
  const last = getLastConsecutiveNot(not);
+ const clone = last.clone({ nodes: [s] });
 
- not.parent.insertAfter(last, last.clone({ nodes: [s] }));
+ clone.rawSpaceBefore = '';
+ not.parent.insertAfter(last, clone);
  }
 }