From 656963e72a61473a5347c77fd9d4071f54a97254 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Wed, 15 May 2024 09:09:50 +0200 Subject: [PATCH] Remove unnecessary `signal` parameter from `removeEventListener` calls This parameter is not necessary, as outlined in https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/removeEventListener#options --- web/app.js | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/web/app.js b/web/app.js index a980be77c..9d454e0c6 100644 --- a/web/app.js +++ b/web/app.js @@ -2044,8 +2044,8 @@ const PDFViewerApplication = { passive: true, signal, }); - mainContainer.removeEventListener("scrollend", scrollend, { signal }); - mainContainer.removeEventListener("blur", scrollend, { signal }); + mainContainer.removeEventListener("scrollend", scrollend); + mainContainer.removeEventListener("blur", scrollend); }; const scroll = () => { if (this._isCtrlKeyDown) { @@ -2059,10 +2059,7 @@ const PDFViewerApplication = { return; } - mainContainer.removeEventListener("scroll", scroll, { - passive: true, - signal, - }); + mainContainer.removeEventListener("scroll", scroll, { passive: true }); this._isScrolling = true; mainContainer.addEventListener("scrollend", scrollend, { signal }); mainContainer.addEventListener("blur", scrollend, { signal });