From 2f127df97b3eb9db0cd916fe8856a6b5902cf992 Mon Sep 17 00:00:00 2001 From: Jonas Jenwald Date: Fri, 1 Jul 2016 14:02:38 +0200 Subject: [PATCH] =?UTF-8?q?Add=20back=20moz-specific=20fullscreen=20suppor?= =?UTF-8?q?t=20code=20for=20=C2=B4MOZCENTRAL`=20builds=20(bug=201268749,?= =?UTF-8?q?=20PR=207273=20follow-up)?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit In PR 7273, we simplified the `MOZCENTRAL` specific check for fullscreen support. Unfortunately I've just, by coincidence, found out about https://bugzilla.mozilla.org/show_bug.cgi?id=1268749, which disabled the unprefixed fullscreen support in release versions of Firefox. If we do nothing, this will lead to Presentation Mode being unavailable in future Firefox releases. This patch should fix things for now, and I'm afraid that we'll need to uplift it to Firefox 49 as well. --- web/app.js | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/web/app.js b/web/app.js index 4a68d99cb..ddd852533 100644 --- a/web/app.js +++ b/web/app.js @@ -416,7 +416,8 @@ var PDFViewerApplication = { get supportsFullscreen() { //#if MOZCENTRAL -// var support = document.fullscreenEnabled === true; +// var support = document.fullscreenEnabled === true || +// document.mozFullScreenEnabled === true; //#else var doc = document.documentElement; var support = !!(doc.requestFullscreen || doc.mozRequestFullScreen ||