Merge pull request #200 from catdad/#195-dedupe-resize-events

registering only a single resize event
catdad · Sep 28, 2023 · fb75303 · fb75303
2 parents 2c028ea + a07e811
commit fb75303
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/confetti.js b/src/confetti.js
@@ -435,6 +435,7 @@
   function confettiCannon(canvas, globalOpts) {
     var isLibCanvas = !canvas;
     var allowResize = !!prop(globalOpts || {}, 'resize');
+    var hasResizeEventRegistered = false;
     var globalDisableForReducedMotion = prop(globalOpts, 'disableForReducedMotion', Boolean);
     var shouldUseWorker = canUseWorker && !!prop(globalOpts || {}, 'useWorker');
     var worker = shouldUseWorker ? getWorker() : null;
@@ -563,6 +564,7 @@
         animationObj = null;
 
         if (allowResize) {
+          hasResizeEventRegistered = false;
           global.removeEventListener('resize', onResize);
         }
 
@@ -573,7 +575,8 @@
         }
       }
 
-      if (allowResize) {
+      if (allowResize && !hasResizeEventRegistered) {
+        hasResizeEventRegistered = true;
         global.addEventListener('resize', onResize, false);
       }