Merge pull request #167 from ThisNameWasTaken/master

Fix issue #166

__tests__/purgecss.test.js

@@ -350,6 +350,10 @@ describe('keyframes', () => {
     it('removes flash', () => {
         expect(purgecssResult.includes('@keyframes flash')).toBe(false)
     })
+    it('keeps keyframes from animations with multiple keyframes', () => {
+        expect(purgecssResult.includes('@keyframes scale')).toBe(true)
+        expect(purgecssResult.includes('@keyframes spin')).toBe(true)
+    })
 })
 
 describe('pseudo selectors', () => {

__tests__/test_examples/keyframes/keyframes.css

@@ -24,4 +24,28 @@
 
 .flash {
     animation: flash
-}
+}
+
+@keyframes scale {
+    from {
+        transform: scale(1);
+    }
+
+    to {
+        transform: scale(2);
+    }
+}
+
+@keyframes spin {
+    from {
+        transform: rotate(0deg);
+    }
+
+    to {
+        transform: rotate(360deg);
+    }
+}
+
+.scale-spin {
+    animation: spin 300ms linear infinite forwards,scale 300ms linear infinite alternate;
+}

__tests__/test_examples/keyframes/keyframes.html

@@ -1,2 +1,5 @@
 <div class="bounce">
 </div>
+
+<div class="scale-spin">
+</div>

src/index.js

@@ -327,7 +327,7 @@ class Purgecss {
             for (const { prop, value } of node.nodes) {
                 if (this.options.keyframes) {
                     if (prop === 'animation' || prop === 'animation-name') {
-                        for (const word of value.split(' ')) {
+                        for (const word of value.split(/[\s,]+/)) {
                             this.usedAnimations.add(word)
                         }
                     }