From 28e34c1c093815aa0243581a17d9c2ecf13b3859 Mon Sep 17 00:00:00 2001 From: Timothy Lim Date: Wed, 14 Dec 2016 16:21:42 +0100 Subject: [PATCH] Simplify fullscreenSupported HFP-339 --- js/h5p.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/js/h5p.js b/js/h5p.js index 559b80b..7e5c068 100644 --- a/js/h5p.js +++ b/js/h5p.js @@ -92,7 +92,7 @@ H5P.init = function (target) { * fullscreen, and the semi-fullscreen solution doesn't work when embedded. * @type {boolean} */ - H5P.fullscreenSupported = (H5P.isFramed && H5P.externalEmbed !== false) ? ((document.fullscreenEnabled || document.webkitFullscreenEnabled || document.mozFullScreenEnabled) ? true : false) : true; + H5P.fullscreenSupported = !(H5P.isFramed && H5P.externalEmbed !== false) || !!(document.fullscreenEnabled || document.webkitFullscreenEnabled || document.mozFullScreenEnabled); // We should consider document.msFullscreenEnabled when they get their // element sizing corrected. Ref. https://connect.microsoft.com/IE/feedback/details/838286/ie-11-incorrectly-reports-dom-element-sizes-in-fullscreen-mode-when-fullscreened-element-is-within-an-iframe } @@ -103,7 +103,7 @@ H5P.init = function (target) { * @deprecated since version 1.11 * @type {boolean} */ - H5P.canHasFullScreen = (H5P.isFramed && H5P.externalEmbed !== false) ? ((document.fullscreenEnabled || document.webkitFullscreenEnabled || document.mozFullScreenEnabled) ? true : false) : true; + H5P.canHasFullScreen = H5P.fullscreenSupported; } // H5Ps added in normal DIV.