sc/source/ui/cctrl/checklistmenu.cxx | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-)
New commits: commit 2d2b08eb1c63e1d12d2dd12abf97e34f56e921c6 Author: Caolán McNamara <caol...@redhat.com> AuthorDate: Tue Sep 3 09:41:12 2019 +0100 Commit: Caolán McNamara <caol...@redhat.com> CommitDate: Tue Sep 3 12:13:06 2019 +0200 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 <caol...@redhat.com> Tested-by: Caolán McNamara <caol...@redhat.com> 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) _______________________________________________ Libreoffice-commits mailing list libreoffice-comm...@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/libreoffice-commits