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

Reply via email to