From 23001910ffb10fe4e65f155d6cb5517e503a6c81 Mon Sep 17 00:00:00 2001 From: "Luke \"Jared\" Bennett" Date: Tue, 31 Jan 2017 14:23:22 +0000 Subject: Fixed collapse-after-update bug --- app/assets/javascripts/right_sidebar.js | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/app/assets/javascripts/right_sidebar.js b/app/assets/javascripts/right_sidebar.js index f3f35f34d59..1b0dee25563 100644 --- a/app/assets/javascripts/right_sidebar.js +++ b/app/assets/javascripts/right_sidebar.js @@ -149,7 +149,7 @@ }; RightSidebar.prototype.setCollapseAfterUpdate = function($block) { - $block.addClass('collapse-after-update'); + $block.closest('.right-sidebar').addClass('collapse-after-update'); $block.data('before-update', $('input', $block).val()); return $('.page-with-sidebar').addClass('with-overlay'); }; @@ -163,11 +163,11 @@ }; RightSidebar.prototype.sidebarDropdownHidden = function($block) { - if ($block.hasClass('collapse-after-update') && $('input', $block).val() !== $block.data('before-update')) { - $block.removeClass('collapse-after-update'); - $('.page-with-sidebar').removeClass('with-overlay'); - return this.toggleSidebar('hide'); - } + if (!$block.closest('.right-sidebar').hasClass('collapse-after-update') || + $('input', $block).val() === $block.data('before-update')) return; + $block.closest('.right-sidebar').removeClass('collapse-after-update'); + $('.page-with-sidebar').removeClass('with-overlay'); + return this.toggleSidebar('hide'); }; RightSidebar.prototype.triggerOpenSidebar = function() { -- cgit v1.2.1