fix(a11y): destroy listeners

uploadcare · Jun 23, 2024 · 497f742 · 497f742
1 parent 8d5ad54
commit 497f742
Showing 1 changed file with 10 additions and 1 deletion.
diff --git a/abstract/a11y.js b/abstract/a11y.js
@@ -15,7 +15,10 @@ import { startKeyUX, hiddenKeyUX, jumpKeyUX, focusGroupKeyUX, pressKeyUX } from
  * @implements {MinimalWindow}
  */
 class ScopedMinimalWindow {
-  /** @private */
+  /**
+   * @private
+   * @type {Map<Function, (event: Event) => void>}
+   */
   _listeners = new Map();
 
   /**
@@ -50,6 +53,7 @@ class ScopedMinimalWindow {
     if (wrappedListener) {
       window.removeEventListener(type, wrappedListener);
     }
+    this._listeners.delete(listener);
   }
 
   get document() {
@@ -67,6 +71,11 @@ class ScopedMinimalWindow {
 
   destroy() {
     this._scope = [];
+    this._listeners.forEach((listener, originalListener) => {
+      window.removeEventListener('keydown', listener);
+      window.removeEventListener('keyup', listener);
+      this._listeners.delete(originalListener);
+    });
   }
 }