Merge pull request #230 from thornbill/fix-scrollbar

Remove unnecessary overflowY changes
jellyfin · Apr 30, 2019 · c491731 · c491731
2 parents 099fe24 + 27c4a00
commit c491731
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 14 deletions.
diff --git a/src/controllers/livetvsuggested.js b/src/controllers/livetvsuggested.js
@@ -284,12 +284,6 @@ define(["layoutManager", "userSettings", "inputManager", "loading", "globalize",
             getTabController(page, index, function (controller) {
                 initialTabIndex = null;
 
-                if (1 === index) {
-                    document.body.classList.add("autoScrollY");
-                } else {
-                    document.body.classList.remove("autoScrollY");
-                }
-
                 if (-1 == renderedTabs.indexOf(index)) {
                     if (1 === index) {
                         renderedTabs.push(index);
@@ -362,7 +356,6 @@ define(["layoutManager", "userSettings", "inputManager", "loading", "globalize",
             if (currentTabController && currentTabController.onHide) {
                 currentTabController.onHide();
             }
-            document.body.classList.remove("autoScrollY");
             inputManager.off(window, onInputCommand);
         });
         view.addEventListener("viewdestroy", function (evt) {

diff --git a/src/controllers/videoosd.js b/src/controllers/videoosd.js
@@ -1169,7 +1169,6 @@ define(["playbackManager", "dom", "inputmanager", "datetime", "itemHelper", "med
             dom.addEventListener(document, window.PointerEvent ? "pointermove" : "mousemove", onPointerMove, {
                 passive: true
             });
-            document.body.classList.add("autoScrollY");
             showOsd();
             inputManager.on(window, onInputCommand);
             dom.addEventListener(window, "keydown", onWindowKeyDown, {
@@ -1190,7 +1189,6 @@ define(["playbackManager", "dom", "inputmanager", "datetime", "itemHelper", "med
             dom.removeEventListener(document, window.PointerEvent ? "pointermove" : "mousemove", onPointerMove, {
                 passive: true
             });
-            document.body.classList.remove("autoScrollY");
             inputManager.off(window, onInputCommand);
             events.off(playbackManager, "playerchange", onPlayerChange);
             releaseCurrentPlayer();

diff --git a/src/css/site.css b/src/css/site.css
@@ -29,16 +29,11 @@ html {
 }
 
 body {
-    overflow-y: scroll !important;
     overflow-x: hidden;
     background-color: transparent !important;
     -webkit-font-smoothing: antialiased
 }
 
-body.autoScrollY {
-    overflow-y: auto !important
-}
-
 .mainAnimatedPage {
     contain: style size !important
 }