Beanz/pdf.js
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Remove unnecessary signal parameter from removeEventListener calls

Browse Source 
This parameter is not necessary, as outlined in https://developer.mozilla.org/en-US/docs/Web/API/EventTarget/removeEventListener#options
This commit is contained in:
Jonas Jenwald 2024-05-15 09:09:50 +02:00
parent 7e5e06b886
commit 656963e72a
1 changed files with 3 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
web/app.js
View File

@ -2044,8 +2044,8 @@ const PDFViewerApplication = {
passive: true, passive: true,
signal, signal,
}); });
mainContainer.removeEventListener("scrollend", scrollend, { signal }); mainContainer.removeEventListener("scrollend", scrollend);
mainContainer.removeEventListener("blur", scrollend, { signal }); mainContainer.removeEventListener("blur", scrollend);
}; };
const scroll = () => { const scroll = () => {
if (this._isCtrlKeyDown) { if (this._isCtrlKeyDown) {
@ -2059,10 +2059,7 @@ const PDFViewerApplication = {
return; return;
} }
mainContainer.removeEventListener("scroll", scroll, { mainContainer.removeEventListener("scroll", scroll, { passive: true });
passive: true,
signal,
});
this._isScrolling = true; this._isScrolling = true;
mainContainer.addEventListener("scrollend", scrollend, { signal }); mainContainer.addEventListener("scrollend", scrollend, { signal });
mainContainer.addEventListener("blur", scrollend, { signal }); mainContainer.addEventListener("blur", scrollend, { signal });