Move more code/methods into BaseViewer
, and simplify the PDFSinglePageViewer._scrollIntoView
method slightly
#10489
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Move additional code/methods into
BaseViewer
and have the extending classes override/extend methods as necessaryThis attempts to provide more "default" methods in the base class, in order to reduce unnecessary duplication and to improve self-documentation of the
BaseViewer
class slightly.The following changes are made (in no particular order):
BaseViewer
implement the_scrollIntoView
method, and extend it as necessary inPDFViewer
/PDFSinglePageViewer
.BaseViewer._resizeBuffer
method, inBaseViewer.update
, since there's only one call-site at this point._isScrollModeHorizontal
inBaseViewer
, and havePDFSinglePageViewer
override it._getVisiblePages
, and havePDFViewer
extend it andPDFSinglePageViewer
override it.Try to simplify the
PDFSinglePageViewer._scrollIntoView
method slightly, by unconditionally ensuring that rendering always occurs