diff --git a/apps/common/main/lib/component/Mixtbar.js b/apps/common/main/lib/component/Mixtbar.js index 72d5e9ed4c..946e32b440 100644 --- a/apps/common/main/lib/component/Mixtbar.js +++ b/apps/common/main/lib/component/Mixtbar.js @@ -61,7 +61,7 @@ define([ }; function onTabDblclick(e) { - var tab = $(e.target).data('tab'); + var tab = $(e.currentTarget).find('> a[data-tab]').data('tab'); if ( this.dblclick_el == tab ) this.fireEvent('change:compact', [tab]); } @@ -235,7 +235,6 @@ define([ onTabClick: function (e) { var me = this; - var $target = $(e.currentTarget); var tab = $target.find('> a[data-tab]').data('tab'); var islone = $target.hasClass('x-lone'); @@ -245,8 +244,12 @@ define([ // me.fireEvent('') } else if ( $target.hasClass('active') ) { - me.collapse(); + !me._timerSetTab && me.collapse(); } else { + me._timerSetTab = true; + setTimeout(function(){ + me._timerSetTab = false; + }, 500); me.setTab(tab); me.processPanelVisible(null, true); } diff --git a/apps/spreadsheeteditor/main/app/controller/DocumentHolder.js b/apps/spreadsheeteditor/main/app/controller/DocumentHolder.js index e0b4360f4f..079dafa955 100644 --- a/apps/spreadsheeteditor/main/app/controller/DocumentHolder.js +++ b/apps/spreadsheeteditor/main/app/controller/DocumentHolder.js @@ -1701,7 +1701,7 @@ define([ documentHolder.mnuChartEdit.chartInfo = elValue; ischartmenu = true; has_chartprops = true; - } if ( elValue.asc_getSlicerProperties() ) { + } else if ( elValue.asc_getSlicerProperties() ) { documentHolder.mnuSlicerAdvanced.imageInfo = elValue; isslicermenu = true; } else {