Merge pull request #16 from timbofield/master

Remove event listeners on unmount

Author: gilbox

src/index.js

@@ -35,7 +35,12 @@ function factory(options) {
       } else {
         spark(element, () => element, this.props.timeline, this.props);
       }
+    },
+
+    componentWillUnmount() {
+        spark.cleanup();
     }
+
   });
 
   const SparkScroll = sparkScrollFactory('div');

src/spark.js

@@ -302,10 +302,18 @@ function sparkFactory({animator, formulas, actionProps, setup, invalidateAutomat
     window.addEventListener('resize', onInvalidate, false);
     eventEmitter.on('invalidate', onInvalidate);
 
+    eventEmitter.once('cleanup', function() {
+        window.removeEventListener('scroll', onScroll);
+        window.removeEventListener('resize', onInvalidate);
+        eventEmitter.removeListener('invalidate', onInvalidate)
+    });
+
     // delay parse a frame to allow proxy to render
     animationFrame.request(parseData.bind(null,timeline));
 
   };
+  
+  spark.cleanup = () => eventEmitter.emit('cleanup');
 
   spark.invalidate = () => eventEmitter.emit('invalidate');
 
@@ -400,4 +408,4 @@ const _sparkActionProps = {
   }
 };
 
-module.exports = sparkFactory;
+module.exports = sparkFactory;