Merge pull request #13801 from Snuffleupagus/AnnotationLayer-check-navigator
Access `navigator` safely in the `src/display/annotation_layer.js` file
This commit is contained in:
commit
e51cbe63bf
@ -24,6 +24,7 @@ import {
|
|||||||
AnnotationBorderStyleType,
|
AnnotationBorderStyleType,
|
||||||
AnnotationType,
|
AnnotationType,
|
||||||
assert,
|
assert,
|
||||||
|
shadow,
|
||||||
stringToPDFString,
|
stringToPDFString,
|
||||||
unreachable,
|
unreachable,
|
||||||
Util,
|
Util,
|
||||||
@ -359,6 +360,15 @@ class AnnotationElement {
|
|||||||
render() {
|
render() {
|
||||||
unreachable("Abstract method `AnnotationElement.render` called");
|
unreachable("Abstract method `AnnotationElement.render` called");
|
||||||
}
|
}
|
||||||
|
|
||||||
|
static get platform() {
|
||||||
|
const platform = typeof navigator !== "undefined" ? navigator.platform : "";
|
||||||
|
|
||||||
|
return shadow(this, "platform", {
|
||||||
|
isWin: platform.includes("Win"),
|
||||||
|
isMac: platform.includes("Mac"),
|
||||||
|
});
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
class LinkAnnotationElement extends AnnotationElement {
|
class LinkAnnotationElement extends AnnotationElement {
|
||||||
@ -539,10 +549,8 @@ class WidgetAnnotationElement extends AnnotationElement {
|
|||||||
}
|
}
|
||||||
|
|
||||||
_getKeyModifier(event) {
|
_getKeyModifier(event) {
|
||||||
return (
|
const { isWin, isMac } = AnnotationElement.platform;
|
||||||
(navigator.platform.includes("Win") && event.ctrlKey) ||
|
return (isWin && event.ctrlKey) || (isMac && event.metaKey);
|
||||||
(navigator.platform.includes("Mac") && event.metaKey)
|
|
||||||
);
|
|
||||||
}
|
}
|
||||||
|
|
||||||
_setEventListener(element, baseName, eventName, valueGetter) {
|
_setEventListener(element, baseName, eventName, valueGetter) {
|
||||||
|
Loading…
x
Reference in New Issue
Block a user