From 4ae6132d07cb21a53ba6149bd4e7bba95a105815 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Sun, 13 Oct 2024 11:39:36 +0200 Subject: [PATCH] Remove unnecessary options from `removeEventListener` in the `web/app.js` file Only the `capture` parameter matters when removing an event listener, see https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/removeEventListener#matching_event_listeners_for_removal --- web/app.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/web/app.js b/web/app.js index a6fa45458..9b92254ee 100644 --- a/web/app.js +++ b/web/app.js @@ -2117,7 +2117,7 @@ const PDFViewerApplication = { return; } - mainContainer.removeEventListener("scroll", scroll, { passive: true }); + mainContainer.removeEventListener("scroll", scroll); this._isScrolling = true; mainContainer.addEventListener("scrollend", scrollend, { signal }); mainContainer.addEventListener("blur", scrollend, { signal });