diff options
author | Jan-Christoph Borchardt <hey@jancborchardt.net> | 2015-03-27 09:37:52 +0100 |
---|---|---|
committer | Morris Jobke <hey@morrisjobke.de> | 2015-06-04 14:31:35 +0200 |
commit | a040a5b08e83b08d98780554cef053914fc01a4b (patch) | |
tree | 75e475a0f4762ba11b020c444fb8c3b35d7b4498 /core | |
parent | eeace88beb588a902c17462dc5ac857c2ec3b629 (diff) | |
download | nextcloud-server-a040a5b08e83b08d98780554cef053914fc01a4b.tar.gz nextcloud-server-a040a5b08e83b08d98780554cef053914fc01a4b.zip |
fix scrutinizer issues
Diffstat (limited to 'core')
-rw-r--r-- | core/js/js.js | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/core/js/js.js b/core/js/js.js index 410d5156842..401295244cf 100644 --- a/core/js/js.js +++ b/core/js/js.js @@ -1194,18 +1194,19 @@ function initCore() { // adjust controls bar width var adjustControlsWidth = function() { if($('#controls').length) { + var controlsWidth; // if there is a scrollbar … if($('#app-content').get(0).scrollHeight > $('#app-content').height()) { if($(window).width() > 768) { - var controlsWidth = $('#content').width() - $('#app-navigation').width() - getScrollBarWidth(); + controlsWidth = $('#content').width() - $('#app-navigation').width() - getScrollBarWidth(); } else { - var controlsWidth = $('#content').width() - getScrollBarWidth(); + controlsWidth = $('#content').width() - getScrollBarWidth(); } } else { // if there is none if($(window).width() > 768) { - var controlsWidth = $('#content').width() - $('#app-navigation').width(); + controlsWidth = $('#content').width() - $('#app-navigation').width(); } else { - var controlsWidth = $('#content').width(); + controlsWidth = $('#content').width(); } } $('#controls').css('width', controlsWidth); @@ -1661,9 +1662,11 @@ function getScrollBarWidth() { var w1 = inner.offsetWidth; outer.style.overflow = 'scroll'; var w2 = inner.offsetWidth; - if (w1 == w2) w2 = outer.clientWidth; + if(w1 === w2) { + w2 = outer.clientWidth; + } document.body.removeChild (outer); return (w1 - w2); -}; +} |