Remove event listeners (#459)

* Remove event listeners before removing DOM elements to prevent events from firing when DOM element is not present * ESLint fix
PaulLeCam · Mar 13, 2018 · 67d4ede · 67d4ede
1 parent 2d24847
commit 67d4ede
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/src/Map.js b/src/Map.js
@@ -336,6 +336,10 @@ export default class Map extends MapComponent<LeafletElement, Props> {
 
   componentWillUnmount() {
     super.componentWillUnmount()
+
+    this.leafletElement.off('move', this.onViewportChange)
+    this.leafletElement.off('moveend', this.onViewportChanged)
+
     // The canvas renderer uses requestAnimationFrame, making a deferred call to a deleted object
     // When preferCanvas is set, use simpler teardown logic
     if (this.props.preferCanvas === true) {