Move all PDFJS display/ usages into global.js file. #7126
Conversation
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
2ef3f38 to
d7a9d3c
Compare
| deprecated('PDFJS.openExternalLinksInNewWindow, please use ' + | ||
| '"PDFJS.externalLinkTarget = PDFJS.LinkTarget.BLANK" instead.'); | ||
| } | ||
| PDFJS.externalLinkTarget = value ? LinkTarget.BLANK : LinkTarget.NONE; |
There was a problem hiding this comment.
Nit: It might not matter too much given that we should completely remove PDFJS.openExternalLinksInNewWindow at some point, but note that in the current code setting PDFJS.openExternalLinksInNewWindow = true; is respected if and only if PDFJS.externalLinkTarget === LinkTarget.NONE, and not unconditionally like this (since PDFJS.externalLinkTarget should always take precedence).
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
d7a9d3c to
409658f
Compare
| link.target = LinkTargetStringMap[PDFJS.externalLinkTarget]; | ||
| var target = params.target; | ||
| if (typeof target === 'undefined') { | ||
| target = getDefaultSetting('externalLinkTarget'); |
There was a problem hiding this comment.
PDFJS.externalLinkTarget is an enumeration value, so LinkTargetStringMap still needs to be used, otherwise link.target isn't set correctly.
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
409658f to
c9f79a2
Compare
web/viewer.js
Outdated
| this.pdfSidebar.onToggled = this.forceRendering.bind(this); | ||
|
|
||
| var self = this; | ||
| var PDFJS = pdfjsLib.PDFJS; |
There was a problem hiding this comment.
Why can't we get rid of this? If we keep it around, we should also use it, e.g. at https://github.com/mozilla/pdf.js/pull/7126/files#diff-dd42e9b2d2a652b8e312efa567698aa6R832
There was a problem hiding this comment.
it's used below a lot.
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
c9f79a2 to
fbafbc0
Compare
src/display/api.js
Outdated
| this.commonObjs = new PDFObjects(); | ||
| this.fontLoader = new FontLoader(loadingTask.docId); | ||
| this.fontLoader = new FontLoader(loadingTask.docId, { | ||
| isEvalSupported: getDefaultSetting('isEvalSupported'), |
There was a problem hiding this comment.
In https://github.com/mozilla/pdf.js/pull/7126/files#diff-c0a7c347be94f54892dea2ead8b3df4cR39, there's no parameter that takes this isEvalSupported argument. And as far as I can tell, isEvalSupported is not used within FontLoader, so is this line really necessary?
There was a problem hiding this comment.
It is not. Reverting the change.
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
fbafbc0 to
9c5a27e
Compare
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
9c5a27e to
e2d6425
Compare
|
/botio-linux preview |
From: Bot.io (Linux)ReceivedCommand cmd_preview from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/baa7951d81369ff/output.txt |
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/baa7951d81369ff/output.txt Total script time: 1.02 mins Published |
|
/botio test |
From: Bot.io (Linux)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.21.233.14:8877/5dc09b9c2d459cc/output.txt |
From: Bot.io (Windows)ReceivedCommand cmd_test from @timvandermeij received. Current queue size: 0 Live output at: http://107.22.172.223:8877/ae6fdf2adff0b68/output.txt |
From: Bot.io (Windows)SuccessFull output at http://107.22.172.223:8877/ae6fdf2adff0b68/output.txt Total script time: 20.35 mins
|
From: Bot.io (Linux)SuccessFull output at http://107.21.233.14:8877/5dc09b9c2d459cc/output.txt Total script time: 22.61 mins
|
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
5b7bccd to
26c577a
Compare
yurydelendik
force-pushed
the
rm-pdfjs-display
branch
from
26c577a to
1e4886a
Compare
5 participants
The dom_utils.js may use global PDFJS to pull values (if the object exists), see getDefaultSetting.
Also, exposed more functions/classes for generic viewer and examples and changed PDFJS object access for these.
Partially addresses #6779