Marmelator/pdf.js.mirror
1
0
Fork 0
mirror of https://github.com/mozilla/pdf.js.git synced 2026-07-07 07:35:46 +02:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #20704 from Snuffleupagus/getPageAdvance-getOrInsert

Browse Source 
Use `Map.prototype.getOrInsert()` in the `_getPageAdvance` method
This commit is contained in:
Jonas Jenwald 2026-02-22 10:32:44 +01:00 committed by GitHub
commit e62a990ae2
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
1 changed files with 4 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

15
web/pdf_viewer.js
View File

@ -2270,10 +2270,7 @@ class PDFViewer {
this.update(); this.update();
} }
/** #getPageAdvance(currentPageNumber, previous = false) {
* @private
*/
_getPageAdvance(currentPageNumber, previous = false) {
switch (this._scrollMode) { switch (this._scrollMode) {
case ScrollMode.WRAPPED: { case ScrollMode.WRAPPED: {
const { views } = this._getVisiblePages(), const { views } = this._getVisiblePages(),
@ -2284,11 +2281,7 @@ class PDFViewer {
if (percent === 0 || widthPercent < 100) { if (percent === 0 || widthPercent < 100) {
continue; continue;
} }
let yArray = pageLayout.get(y); pageLayout.getOrInsert(y, []).push(id);
if (!yArray) {
pageLayout.set(y, (yArray ||= []));
}
yArray.push(id);
} }
// Find the row of the current page. // Find the row of the current page.
for (const yArray of pageLayout.values()) { for (const yArray of pageLayout.values()) {
@ -2379,7 +2372,7 @@ class PDFViewer {
return false; return false;
} }
const advance = const advance =
this._getPageAdvance(currentPageNumber, /* previous = */ false) || 1; this.#getPageAdvance(currentPageNumber, /* previous = */ false) || 1;
this.currentPageNumber = Math.min(currentPageNumber + advance, pagesCount); this.currentPageNumber = Math.min(currentPageNumber + advance, pagesCount);
return true; return true;
@ -2396,7 +2389,7 @@ class PDFViewer {
return false; return false;
} }
const advance = const advance =
this._getPageAdvance(currentPageNumber, /* previous = */ true) || 1; this.#getPageAdvance(currentPageNumber, /* previous = */ true) || 1;
this.currentPageNumber = Math.max(currentPageNumber - advance, 1); this.currentPageNumber = Math.max(currentPageNumber - advance, 1);
return true; return true;