cotugno/jellyfin-web
2
Fork 0
mirror of https://github.com/jellyfin/jellyfin-web.git synced 2024-11-17 19:08:18 -07:00
Code Issues Packages Projects Releases Wiki Activity

Apply suggestions from code review

Browse Source
This commit is contained in:
Dmitry Lyzo 2019-11-27 14:30:56 +03:00
parent 3752db775b
commit c677708819
1 changed files with 5 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

14
src/components/scrollManager.js
View File

@ -12,7 +12,6 @@ define(["dom", "browser", "layoutManager"], function (dom, browser, layoutManage
var Epsilon = 1e-6;
// FIXME: Need to scroll to top of page to fully show the top menu. This can be solved by some marker of top most elements or their containers
var _minimumScrollY = 0;
/**
* Returns minimum vertical scroll.
* Scroll less than that value will be zeroed.
@ -20,13 +19,11 @@ define(["dom", "browser", "layoutManager"], function (dom, browser, layoutManage
* @return {number} minimum vertical scroll
*/
function minimumScrollY() {
if (_minimumScrollY === 0) {
var topMenu = document.querySelector(".headerTop");
if (topMenu) {
_minimumScrollY = topMenu.clientHeight;
}
var topMenu = document.querySelector(".headerTop");
if (topMenu) {
return topMenu.clientHeight;
}
return _minimumScrollY;
return 0;
}
var supportsSmoothScroll = "scrollBehavior" in document.documentElement.style;
@ -333,8 +330,7 @@ define(["dom", "browser", "layoutManager"], function (dom, browser, layoutManage
if (k === 1) {
resetScrollTimer();
xScroller.scrollLeft = scrollX;
yScroller.scrollTop = scrollY;
builtinScroll(xScroller, scrollX, yScroller, scrollY, false);
return;
}