fix: ignore subselectors

patrickhulce · Mar 5, 2017 · 02d2074 · 02d2074
1 parent ed966de
commit 02d2074
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 1 deletion.
diff --git a/lib/parser/gonzales.js b/lib/parser/gonzales.js
@@ -4,7 +4,12 @@ const gonzales = require('gonzales-pe')
 class GonzalesParser {
   static getSelectorTokensInRuleset(parsed) {
     const selectors = []
-    parsed.traverseByType('selector', node => {
+    parsed.traverseByType('selector', (node, index, parent) => {
+      // don't target on subselectors like :not([disabled])
+      if (parent.type === 'arguments') {
+        return
+      }
+
       const tokens = GonzalesParser.getTokensInSelector(node)
       selectors.push({node, tokens})
     })

diff --git a/test/fixtures/content.css b/test/fixtures/content.css
@@ -25,3 +25,7 @@ html .something > foobar[something=x] #xb {
 .unused, .something {
   color: white;
 }
+
+madeup:not([role]) {
+  color: white;
+}
diff --git a/test/nuke.test.js b/test/nuke.test.js
@@ -19,6 +19,7 @@ describe('nuke.js', () => {
       expect(result).to.not.contain('foobar[something=x]')
       expect(result).to.not.contain('.something3')
       expect(result).to.not.contain('.foo-bar-3')
+      expect(result).to.not.contain('madeup:not')
     })
 
     it('should remove empty media sets', () => {