From 2d2b08eb1c63e1d12d2dd12abf97e34f56e921c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= Date: Tue, 3 Sep 2019 09:41:12 +0100 Subject: [PATCH] Resolves: tdf#127168 dismiss menu before executing sort MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit 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 Tested-by: Caolán McNamara --- sc/source/ui/cctrl/checklistmenu.cxx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) 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)