From 5f07a8e75192d7f5d0db05ec0419883421e0b95b Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Kasper=20Rynning-T=C3=B8nnesen?= Date: Fri, 31 Aug 2018 11:43:44 +0200 Subject: [PATCH] Fixed issue where clicking the video-container on mobile did not hide the player --- server/public/assets/js/player.js | 27 +++++++++++++-------------- 1 file changed, 13 insertions(+), 14 deletions(-) diff --git a/server/public/assets/js/player.js b/server/public/assets/js/player.js index e4c261f3..d2970f1c 100755 --- a/server/public/assets/js/player.js +++ b/server/public/assets/js/player.js @@ -143,20 +143,17 @@ var Player = { Helper.css("#player", "visibility", "visible"); } if(!embed && !client && window.location.pathname != "/") { + if(Helper.mobilecheck()) { + Helper.css("#player", "display", "block"); + Helper.toggleClass(".video-container", "click-through"); + Helper.toggleClass(".page-footer", "padding-bottom-extra"); + } resizePlaylistPlaying(newState.data == YT.PlayerState.PLAYING || newState.data == YT.PlayerState.BUFFERING); } if(embed && !autoplay) autoplay = true; - //if(!window.MSStream) { - Helper.css("#player", "opacity", "1"); - //if(!Helper.mobilecheck()) { - Helper.css("#channel-load", "display", "none"); + Helper.css("#player", "opacity", "1"); + Helper.css("#channel-load", "display", "none"); - //} - //} - /*if(Helper.mobilecheck()) { - Helper.css("#player", "display", "block"); - Helper.css("#player", "pointer-events", "all"); - }*/ Helper.addClass("#player", "pointer-events-all-mobile"); Helper.removeClass("#video-container", "click-through"); Helper.addClass("#player", "small-display"); @@ -176,6 +173,7 @@ var Player = { Helper.toggleClass("#play", "hide"); if(document.getElementById("pause").className.split(" ").length == 2) Helper.toggleClass("#pause", "hide"); + } buffering = false; if((paused || was_stopped) && !offline) { @@ -206,14 +204,15 @@ var Player = { if(window.location.pathname != "/") Playercontrols.play_pause_show(); mobile_beginning = true; if(!embed && !client && window.location.pathname != "/") { + if(Helper.mobilecheck()) { + Helper.css("#player", "display", "none"); + Helper.toggleClass(".video-container", "click-through"); + Helper.toggleClass(".page-footer", "padding-bottom-extra"); + } resizePlaylistPlaying(newState.data == YT.PlayerState.PLAYING || newState.data == YT.PlayerState.BUFFERING); } } } - /*if(Helper.mobilecheck()) { - Helper.css("#player", "display", "none"); - Helper.css("#player", "pointer-events", "none"); - }*/ Helper.removeClass("#player", "pointer-events-all-mobile"); Helper.addClass("#video-container", "click-through"); Helper.removeClass("#player", "small-display");