remove ignore comment

Author: Ffloriel

__tests__/purgecssDefault.test.js

@@ -204,13 +204,21 @@ describe('purge methods with files and default extractor', () => {
     })
 
     describe('ignore comment', () => {
-        it('ignores h1', () => {
+        let purgecssResult
+        beforeAll(() => {
             const purgecss = new Purgecss({
                 content: [`${root}ignore_comment/ignore_comment.html`],
                 css: [`${root}ignore_comment/ignore_comment.css`]
             })
-            const result = purgecss.purge()[0].css
-            expect(result.includes('h1')).toBe(true)
+            purgecssResult = purgecss.purge()[0].css
+        })
+        it('ignores h1', () => {
+            expect(purgecssResult.includes('h1')).toBe(true)
+        })
+
+        it('removes the comment', () => {
+            console.log(purgecssResult)
+            expect(purgecssResult.includes('/* purgecss ignore */')).toBe(false)
         })
     })
 
@@ -232,6 +240,12 @@ describe('purge methods with files and default extractor', () => {
         it('removes h4', () => {
             expect(purgecssResult.includes('h4')).toBe(false)
         })
+
+        it('removes the comments', () => {
+            console.log(purgecssResult)
+            expect(purgecssResult.includes('/* purgecss start ignore */')).toBe(false)
+            expect(purgecssResult.includes('/* purgecss end ignore */')).toBe(false)
+        })
     })
 
     describe('font-face', () => {
@@ -289,7 +303,7 @@ describe('purge methods with files and default extractor', () => {
         })
 
         it('conserves empty attributes', () => {
-            expect(purgecssResult.includes('input[value=""]')).toBe(true);
+            expect(purgecssResult.includes('input[value=""]')).toBe(true)
         })
     })
 

src/index.js

@@ -89,12 +89,12 @@ class Purgecss {
         // Get selectors from content files
         const { content, extractors, css } = this.options
 
-        const fileFormatContents = ((content.filter(o => typeof o === 'string'): Array<any>): Array<
-            string
-        >)
-        const rawFormatContents = ((content.filter(o => typeof o === 'object'): Array<any>): Array<
-            RawContent
-        >)
+        const fileFormatContents = ((content.filter(
+            o => typeof o === 'string'
+        ): Array<any>): Array<string>)
+        const rawFormatContents = ((content.filter(
+            o => typeof o === 'object'
+        ): Array<any>): Array<RawContent>)
 
         const cssFileSelectors = this.extractFileSelector(fileFormatContents, extractors)
         const cssRawSelectors = this.extractRawSelector(rawFormatContents, extractors)
@@ -244,9 +244,10 @@ class Purgecss {
     extractSelectors(content: string, extractor: Object | Function): Set<string> {
         let selectors = new Set()
 
-        const arraySelector = typeof extractor.extract === 'undefined'
-            ? extractor(content)
-            : extractor.extract(content)
+        const arraySelector =
+            typeof extractor.extract === 'undefined'
+                ? extractor(content)
+                : extractor.extract(content)
         if (arraySelector === null) {
             throw new Error(ERROR_EXTRACTER_FAILED)
         }
@@ -271,8 +272,15 @@ class Purgecss {
                 return this.evaluateAtRule(node)
             }
             if (node.type === 'comment') {
-                if (this.isIgnoreAnnotation(node, 'start')) this.ignore = true
-                else if (this.isIgnoreAnnotation(node, 'end')) this.ignore = false
+                if (this.isIgnoreAnnotation(node, 'start')) {
+                    this.ignore = true
+                    // remove ignore annotation
+                    node.remove()
+                } else if (this.isIgnoreAnnotation(node, 'end')) {
+                    this.ignore = false
+                    // remove ignore annotation
+                    node.remove()
+                }
             }
         })
     }
@@ -284,7 +292,15 @@ class Purgecss {
      */
     evaluateRule(node: Object, selectors: Set<string>) {
         const annotation = node.prev()
-        if (this.isIgnoreAnnotation(annotation, 'next') || this.ignore === true) return
+        if (this.ignore) return
+        if (
+            typeof annotation !== 'undefined' &&
+            annotation.type === 'comment' &&
+            this.isIgnoreAnnotation(annotation, 'next')
+        ) {
+            annotation.remove()
+            return
+        }
 
         let keepSelector = true
         node.selector = selectorParser(selectorsParsed => {