diff options
author | Caolán McNamara <caolanm@redhat.com> | 2019-09-03 09:41:12 +0100 |
---|---|---|
committer | Caolán McNamara <caolanm@redhat.com> | 2019-09-03 12:13:06 +0200 |
commit | 2d2b08eb1c63e1d12d2dd12abf97e34f56e921c6 (patch) | |
tree | f46ea14ec95426d1f30f66b4efec6cf9ab818339 | |
parent | a06b83ff927d8973c8b4a2de9a0c7ce5cd4b1f96 (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.cxx | 3 |
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) |