[Libreoffice-commits] core.git: Branch 'libreoffice-7-0' - cui/uiconfig

Markus Mohrhard (via logerrit) logerrit at kemper.freedesktop.org
Tue Aug 4 01:48:26 UTC 2020


 cui/uiconfig/ui/movemenu.ui |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit 23397ecc1ee69462fbe0639c5ea105e2c0ec9a45
Author:     Markus Mohrhard <markus.mohrhard at googlemail.com>
AuthorDate: Sat Aug 1 23:01:52 2020 +0800
Commit:     Adolfo Jayme Barrientos <fitojb at ubuntu.com>
CommitDate: Tue Aug 4 03:47:49 2020 +0200

    correctly mark the "New menu" dialog as modal
    
    The dialog is called from a modal dialog which means a modeless dialog
    does not make sense. Some of our internal dialog tracking code is confused
    by that case which confused the UI tests.
    
    Change-Id: I705877e8a751c55bfbd00ddeaf18ab86c95321c3
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99944
    Tested-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    Reviewed-by: Markus Mohrhard <markus.mohrhard at googlemail.com>
    (cherry picked from commit 3258603e8850900e04cecffb1113b833c4c76fd9)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/99929
    Tested-by: Jenkins
    Reviewed-by: Adolfo Jayme Barrientos <fitojb at ubuntu.com>

diff --git a/cui/uiconfig/ui/movemenu.ui b/cui/uiconfig/ui/movemenu.ui
index 7bb5be63309c..18a0712e67c1 100644
--- a/cui/uiconfig/ui/movemenu.ui
+++ b/cui/uiconfig/ui/movemenu.ui
@@ -25,6 +25,7 @@
     <property name="border_width">6</property>
     <property name="title" translatable="yes" context="movemenu|MoveMenuDialog">New Menu</property>
     <property name="type_hint">dialog</property>
+    <property name="modal">True</property>
     <child>
       <placeholder/>
     </child>


More information about the Libreoffice-commits mailing list