Merge pull request #25 from retejs/doritosxxx-patch-1

fix: memory leak
retejs · Oct 12, 2024 · 35f4a8b · 35f4a8b
2 parents 148ac5d + da13ae5
commit 35f4a8b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/zoom.ts b/src/zoom.ts
@@ -28,8 +28,8 @@ export class Zoom {
 
     window.addEventListener('pointermove', this.move)
     window.addEventListener('pointerup', this.up)
-    window.addEventListener('contextmenu', this.contextmenu)
     window.addEventListener('pointercancel', this.up)
+    window.addEventListener('contextmenu', this.contextmenu)
   }
 
   protected wheel = (e: WheelEvent) => {
@@ -117,5 +117,6 @@ export class Zoom {
     window.removeEventListener('pointermove', this.move)
     window.removeEventListener('pointerup', this.up)
     window.removeEventListener('pointercancel', this.up)
+    window.removeEventListener('contextmenu', this.contextmenu)
   }
 }