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

Remove CSS variables feature-testing from PDFSidebarResizer

Browse Source 
CSS variables are now supported in all reasonably modern browsers, according to:
 - https://developer.mozilla.org/en-US/docs/Web/CSS/Using_CSS_custom_properties#Browser_compatibility
 - https://caniuse.com/css-variables
This commit is contained in:
Jonas Jenwald 2020-09-09 13:29:44 +02:00
parent 2594f0c738
commit 01c1d87171
2 changed files with 2 additions and 25 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

25
web/pdf_sidebar_resizer.js
View File

@ -34,7 +34,6 @@ class PDFSidebarResizer {
* @param {IL10n} l10n - Localization service. * @param {IL10n} l10n - Localization service.
*/ */
constructor(options, eventBus, l10n = NullL10n) { constructor(options, eventBus, l10n = NullL10n) {
this.enabled = false;
this.isRTL = false; this.isRTL = false;
this.sidebarOpen = false; this.sidebarOpen = false;
this.doc = document.documentElement; this.doc = document.documentElement;
@ -45,24 +44,8 @@ class PDFSidebarResizer {
this.outerContainer = options.outerContainer; this.outerContainer = options.outerContainer;
this.resizer = options.resizer; this.resizer = options.resizer;
this.eventBus = eventBus; this.eventBus = eventBus;
this.l10n = l10n;
if ( l10n.getDirection().then(dir => {
(typeof PDFJSDev === "undefined" || !PDFJSDev.test("MOZCENTRAL")) &&
(typeof CSS === "undefined" ||
typeof CSS.supports !== "function" ||
!CSS.supports(SIDEBAR_WIDTH_VAR, `calc(-1 * ${SIDEBAR_MIN_WIDTH}px)`))
) {
console.warn(
"PDFSidebarResizer: " +
"The browser does not support resizing of the sidebar."
);
return;
}
this.enabled = true;
this.resizer.classList.remove("hidden"); // Show the resizer DOM element.
this.l10n.getDirection().then(dir => {
this.isRTL = dir === "rtl"; this.isRTL = dir === "rtl";
}); });
this._addEventListeners(); this._addEventListeners();
@ -83,9 +66,6 @@ class PDFSidebarResizer {
* returns {boolean} Indicating if the sidebar width was updated. * returns {boolean} Indicating if the sidebar width was updated.
*/ */
_updateWidth(width = 0) { _updateWidth(width = 0) {
if (!this.enabled) {
return false;
}
// Prevent the sidebar from becoming too narrow, or from occupying more // Prevent the sidebar from becoming too narrow, or from occupying more
// than half of the available viewer width. // than half of the available viewer width.
const maxWidth = Math.floor(this.outerContainerWidth / 2); const maxWidth = Math.floor(this.outerContainerWidth / 2);
@ -134,9 +114,6 @@ class PDFSidebarResizer {
* @private * @private
*/ */
_addEventListeners() { _addEventListeners() {
if (!this.enabled) {
return;
}
const _boundEvents = this._boundEvents; const _boundEvents = this._boundEvents;
_boundEvents.mouseMove = this._mouseMove.bind(this); _boundEvents.mouseMove = this._mouseMove.bind(this);
_boundEvents.mouseUp = this._mouseUp.bind(this); _boundEvents.mouseUp = this._mouseUp.bind(this);

2
web/viewer.html
View File

@ -101,7 +101,7 @@ See https://github.com/adobe-type-tools/cmap-resources
<div id="layersView" class="hidden"> <div id="layersView" class="hidden">
</div> </div>
</div> </div>
<div id="sidebarResizer" class="hidden"></div> <div id="sidebarResizer"></div>
</div> <!-- sidebarContainer --> </div> <!-- sidebarContainer -->
<div id="mainContainer"> <div id="mainContainer">