From 7173be59c6e7f16d81c353affa672e3e1016ea50 Mon Sep 17 00:00:00 2001
From: Maurits Lourens <maurits.lourens@gmail.com>
Date: Mon, 26 Jul 2021 23:53:11 +0200
Subject: [PATCH] remove event listeners prevents memory leaks

---
 .../design/AppPreview/CurrentItemPreview.svelte        | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

diff --git a/packages/builder/src/components/design/AppPreview/CurrentItemPreview.svelte b/packages/builder/src/components/design/AppPreview/CurrentItemPreview.svelte
index a6d186817c..7c0be0e067 100644
--- a/packages/builder/src/components/design/AppPreview/CurrentItemPreview.svelte
+++ b/packages/builder/src/components/design/AppPreview/CurrentItemPreview.svelte
@@ -1,5 +1,5 @@
 <script>
-  import { onMount } from "svelte"
+  import { onMount, onDestroy } from "svelte"
   import { store, currentAsset } from "builderStore"
   import iframeTemplate from "./iframeTemplate"
   import { Screen } from "builderStore/store/screenTemplates/utils/Screen"
@@ -120,6 +120,14 @@
     })
   })
 
+  // remove all iframe event listeners on component destroy
+  onDestroy(() => {
+    iframe.contentWindow.removeEventListener("ready")
+    iframe.contentWindow.removeEventListener("error")
+    iframe.contentWindow.removeEventListener("bb-event")
+    iframe.contentWindow.removeEventListener("keydown")
+  })
+
   const confirmDeleteComponent = (componentId) => {
     idToDelete = componentId
     confirmDeleteDialog.show()