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

Try to simplify the PDFSinglePageViewer._scrollIntoView method slightly, by unconditionally ensuring that rendering always occurs

Browse Source
This commit is contained in:
Jonas Jenwald 2019-01-23 09:13:49 +01:00
parent 06cda4c2e7
commit 48e4adf770
1 changed files with 7 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

20
web/pdf_single_page_viewer.js
View File

@ -39,6 +39,7 @@ class PDFSinglePageViewer extends BaseViewer {
super._resetView();
this._previousPageNumber = 1;
this._shadowViewer = document.createDocumentFragment();
this._updateScrollDown = null;
}
_ensurePageViewVisible() {
@ -82,9 +83,12 @@ class PDFSinglePageViewer extends BaseViewer {
if (pageNumber) { // Ensure that `this._currentPageNumber` is correct.
this._setCurrentPageNumber(pageNumber);
}
let scrolledDown = this._currentPageNumber >= this._previousPageNumber;
let previousLocation = this._location;
const scrolledDown = this._currentPageNumber >= this._previousPageNumber;
this._ensurePageViewVisible();
// Ensure that rendering always occurs, to avoid showing a blank page,
// even if the current position doesn't change when the page is scrolled.
this.update();
super._scrollIntoView({ pageDiv, pageSpot, pageNumber, });
@ -92,18 +96,8 @@ class PDFSinglePageViewer extends BaseViewer {
// scroll direction during the next `this._scrollUpdate` invocation.
this._updateScrollDown = () => {
this.scroll.down = scrolledDown;
delete this._updateScrollDown;
this._updateScrollDown = null;
};
// If the scroll position doesn't change as a result of the `scrollIntoView`
// call, ensure that rendering always occurs to avoid showing a blank page.
setTimeout(() => {
if (this._location === previousLocation) {
if (this._updateScrollDown) {
this._updateScrollDown();
}
this.update();
}
}, 0);
}
_getVisiblePages() {