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

Merge pull request #9815 from Snuffleupagus/rm-disableFullscreen-IE-embedded

Browse Source 
Remove the `disableFullscreen` hack for embedded IE 11 (issue 9585)
This commit is contained in:
Tim van der Meij 2018-06-14 23:17:57 +02:00 committed by GitHub
commit 73843865b8
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
3 changed files with 0 additions and 18 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
web/app.js
View File

@ -553,10 +553,6 @@ let PDFViewerApplication = {
support = false;
}
}
if (support && AppOptions.get('disableFullscreen') === true) {
support = false;
}
return shadow(this, 'supportsFullscreen', support);
},

5
web/app_options.js
View File

@ -43,11 +43,6 @@ const defaultOptions = {
value: '',
kind: OptionKind.VIEWER,
},
disableFullscreen: {
/** @type {boolean} */
value: viewerCompatibilityParams.disableFullscreen || false,
kind: OptionKind.VIEWER,
},
disableHistory: {
/** @type {boolean} */
value: false,

9
web/viewer_compatibility.js
View File

@ -18,17 +18,8 @@ if (typeof PDFJSDev === 'undefined' || PDFJSDev.test('GENERIC')) {
const userAgent =
(typeof navigator !== 'undefined' && navigator.userAgent) || '';
const isAndroid = /Android/.test(userAgent);
const isIE = /Trident/.test(userAgent);
const isIOS = /\b(iPad|iPhone|iPod)(?=;)/.test(userAgent);
// Disable fullscreen support for certain problematic configurations.
// Support: IE11+ (when embedded).
(function checkFullscreenSupport() {
if (isIE && window.parent !== window) {
compatibilityParams.disableFullscreen = true;
}
})();
// Limit canvas size to 5 mega-pixels on mobile.
// Support: Android, iOS
(function checkCanvasSizeLimitation() {