From 97d48edcdf1fd8d90286268bde494a2e069347ca Mon Sep 17 00:00:00 2001 From: XhmikosR Date: Tue, 2 Jun 2020 18:50:19 +0300 Subject: [PATCH] Use an object when setting multiple CSS properties --- build/js/ControlSidebar.js | 6 ++++-- build/js/Dropdown.js | 24 ++++++++++++++++-------- 2 files changed, 20 insertions(+), 10 deletions(-) diff --git a/build/js/ControlSidebar.js b/build/js/ControlSidebar.js index df6e62215..0435e503d 100644 --- a/build/js/ControlSidebar.js +++ b/build/js/ControlSidebar.js @@ -176,8 +176,10 @@ class ControlSidebar { } if (positions.top === 0 && positions.bottom === 0) { - $(Selector.CONTROL_SIDEBAR).css('bottom', heights.footer) - $(Selector.CONTROL_SIDEBAR).css('top', heights.header) + $(Selector.CONTROL_SIDEBAR).css({ + bottom: heights.footer, + top: heights.header + }) $(Selector.CONTROL_SIDEBAR + ', ' + Selector.CONTROL_SIDEBAR + ' ' + Selector.CONTROL_SIDEBAR_CONTENT).css('height', heights.window - (heights.header + heights.footer)) } else if (positions.bottom <= heights.footer) { if (footerFixed === false) { diff --git a/build/js/Dropdown.js b/build/js/Dropdown.js index 344de8030..ad9dbdd40 100644 --- a/build/js/Dropdown.js +++ b/build/js/Dropdown.js @@ -63,11 +63,15 @@ class Dropdown { } if ($element.hasClass(ClassName.DROPDOWN_RIGHT)) { - $element.css('left', 'inherit') - $element.css('right', 0) + $element.css({ + left: 'inherit', + right: 0 + }) } else { - $element.css('left', 0) - $element.css('right', 'inherit') + $element.css({ + left: 0, + right: 'inherit' + }) } const offset = $element.offset() @@ -75,11 +79,15 @@ class Dropdown { const visiblePart = $(window).width() - offset.left if (offset.left < 0) { - $element.css('left', 'inherit') - $element.css('right', (offset.left - 5)) + $element.css({ + left: 'inherit', + right: offset.left - 5 + }) } else if (visiblePart < width) { - $element.css('left', 'inherit') - $element.css('right', 0) + $element.css({ + left: 'inherit', + right: 0 + }) } }