summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorCaolán McNamara <caolanm@redhat.com>2019-09-03 09:41:12 +0100
committerCaolán McNamara <caolanm@redhat.com>2019-09-03 12:13:06 +0200
commit2d2b08eb1c63e1d12d2dd12abf97e34f56e921c6 (patch)
treef46ea14ec95426d1f30f66b4efec6cf9ab818339
parenta06b83ff927d8973c8b4a2de9a0c7ce5cd4b1f96 (diff)
Resolves: tdf#127168 dismiss menu before executing sort
so any error dialog which are parented to the calc window are not hidden by the menu which is destined to be dismissed after the sort anyway Change-Id: I7c49737231901de0e61c290fe161d37809d68424 Reviewed-on: https://gerrit.libreoffice.org/78539 Tested-by: Jenkins Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
-rw-r--r--sc/source/ui/cctrl/checklistmenu.cxx3
1 files changed, 2 insertions, 1 deletions
diff --git a/sc/source/ui/cctrl/checklistmenu.cxx b/sc/source/ui/cctrl/checklistmenu.cxx
index 43553d305f38..413cf36704c0 100644
--- a/sc/source/ui/cctrl/checklistmenu.cxx
+++ b/sc/source/ui/cctrl/checklistmenu.cxx
@@ -452,8 +452,9 @@ void ScMenuFloatingWindow::executeMenuItem(size_t nPos)
// no action is defined.
return;
- maMenuItems[nPos].mpAction->execute();
terminateAllPopupMenus();
+
+ maMenuItems[nPos].mpAction->execute();
}
void ScMenuFloatingWindow::setSelectedMenuItem(size_t nPos, bool bSubMenuTimer, bool bEnsureSubMenu)