diff options
author | Phil Hughes <me@iamphill.com> | 2017-06-23 08:31:20 +0000 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2017-06-23 08:31:20 +0000 |
commit | f76ed8151eb6601681a9e095ec35088e15cefd27 (patch) | |
tree | 128dae05e849fb9f19dfb20179fb0998ae599140 | |
parent | 182db09ea97f7b2389fbb03743b26ce412cca5e7 (diff) | |
parent | c12172ee3374fff0d57b111504773d6e0ad04ee6 (diff) | |
download | gitlab-ce-f76ed8151eb6601681a9e095ec35088e15cefd27.tar.gz |
Merge branch 'fix-sidebar-scroll-jump' into 'master'
Remove layout nav from scroll calculation
Closes #33984
See merge request !12399
-rw-r--r-- | app/assets/javascripts/right_sidebar.js | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/app/assets/javascripts/right_sidebar.js b/app/assets/javascripts/right_sidebar.js index da7c0c5a36c..322162afdb8 100644 --- a/app/assets/javascripts/right_sidebar.js +++ b/app/assets/javascripts/right_sidebar.js @@ -10,8 +10,6 @@ import Cookies from 'js-cookie'; this.$sidebarInner = this.sidebar.find('.issuable-sidebar'); this.$navGitlab = $('.navbar-gitlab'); - this.$layoutNav = $('.layout-nav'); - this.$subScroll = $('.sub-nav-scroll'); this.$rightSidebar = $('.js-right-sidebar'); this.removeListeners(); @@ -215,7 +213,7 @@ import Cookies from 'js-cookie'; }; Sidebar.prototype.setSidebarHeight = function() { - const $navHeight = this.$navGitlab.outerHeight() + this.$layoutNav.outerHeight() + (this.$subScroll ? this.$subScroll.outerHeight() : 0); + const $navHeight = this.$navGitlab.outerHeight(); const diff = $navHeight - $(window).scrollTop(); if (diff > 0) { this.$rightSidebar.outerHeight($(window).height() - diff); |