fix: 🐛 cleanup for eventListeners (#241)

* fix: 🐛 cleanup for eventListeners * chore: 🤖 added changeset
dotlottie · Sep 26, 2023 · b5e2b23 · b5e2b23
1 parent c1d0f54
commit b5e2b23
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 0 deletions.
diff --git a/.changeset/orange-dots-teach.md b/.changeset/orange-dots-teach.md
@@ -0,0 +1,5 @@
+---
+'@dotlottie/react-player': patch
+---
+
+fix: 🐛 cleanup for eventListeners
diff --git a/packages/react-player/src/react-player.tsx b/packages/react-player/src/react-player.tsx
@@ -238,6 +238,14 @@ export const DotLottiePlayer = React.forwardRef<DotLottieCommonPlayer | null, Pr
       dotLottieCommonPlayerRef.current.addEventListener('data_failed', onDataFailed);
       dotLottieCommonPlayerRef.current.addEventListener('complete', onComplete);
       dotLottieCommonPlayerRef.current.addEventListener('loopComplete', onLoopComplete);
+
+      return (): void => {
+        dotLottieCommonPlayerRef.current.removeEventListener('DOMLoaded', onDOMLoaded);
+        dotLottieCommonPlayerRef.current.removeEventListener('data_ready', onDataReady);
+        dotLottieCommonPlayerRef.current.removeEventListener('data_failed', onDataFailed);
+        dotLottieCommonPlayerRef.current.removeEventListener('complete', onComplete);
+        dotLottieCommonPlayerRef.current.removeEventListener('loopComplete', onLoopComplete);
+      };
     }, []);
 
     useEffect(() => {