From 7a7fc1ad34d61a22b6cded76de1dc2e1acfe6fc1 Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Wed, 29 Mar 2023 11:00:31 +0300 Subject: [PATCH] Modal: drop loop (#38368) We only have two elements to iterate over so it doesn't make a lot of sense to use a loop here --- js/src/modal.js | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/js/src/modal.js b/js/src/modal.js index 454d46aea2..c4c4102042 100644 --- a/js/src/modal.js +++ b/js/src/modal.js @@ -139,12 +139,12 @@ class Modal extends BaseComponent { } dispose() { - for (const htmlElement of [window, this._dialog]) { - EventHandler.off(htmlElement, EVENT_KEY) - } + EventHandler.off(window, EVENT_KEY) + EventHandler.off(this._dialog, EVENT_KEY) this._backdrop.dispose() this._focustrap.deactivate() + super.dispose() }