Remove modality of figure options

impact27 · impact27 · commit 38d4065cde14 · 2020-11-17T13:31:18.000+01:00
diff --git a/lib/matplotlib/backends/qt_editor/_formlayout.py b/lib/matplotlib/backends/qt_editor/_formlayout.py
@@ -413,10 +413,11 @@ def get(self):
 class FormDialog(QtWidgets.QDialog):
     """Form Dialog"""
     def __init__(self, data, title="", comment="",
-                 icon=None, parent=None, apply=None):
+                 icon=None, parent=None, apply=None, modal=True):
         super().__init__(parent)
 
         self.apply_callback = apply
+        self.modal = modal
 
         # Form
         if isinstance(data[0][0], (list, tuple)):
@@ -469,6 +470,8 @@ def update_buttons(self):
                 btn.setEnabled(valid)
 
     def accept(self):
+        if not self.modal:
+            self.apply_callback(self.formwidget.get())
         self.data = self.formwidget.get()
         super().accept()
 
@@ -484,7 +487,8 @@ def get(self):
         return self.data
 
 
-def fedit(data, title="", comment="", icon=None, parent=None, apply=None):
+def fedit(data, title="", comment="", icon=None, parent=None, apply=None,
+          modal=True):
     """
     Create form dialog and return result
     (if Cancel button is pressed, return None)
@@ -495,6 +499,7 @@ def fedit(data, title="", comment="", icon=None, parent=None, apply=None):
     icon: QIcon instance
     parent: parent QWidget
     apply: apply callback (function)
+    modal: modality of dialog
 
     datalist: list/tuple of (field_name, field_value)
     datagroup: list/tuple of (datalist *or* datagroup, title, comment)
@@ -511,13 +516,19 @@ def fedit(data, title="", comment="", icon=None, parent=None, apply=None):
       - list/tuple:
           * the first element will be the selected index (or value)
           * the other elements can be couples (key, value) or only values
+    
+    If modal is True, the application will block until the result is selected.
+    If modal is false, the dialog will be shown and the function returns None.
     """
 
     # Create a QApplication instance if no instance currently exists
     # (e.g., if the module is used directly from the interpreter)
     if QtWidgets.QApplication.startingUp():
         _app = QtWidgets.QApplication([])
-    dialog = FormDialog(data, title, comment, icon, parent, apply)
+    dialog = FormDialog(data, title, comment, icon, parent, apply, modal)
+    if not modal:
+        dialog.show()
+        return
     if dialog.exec_():
         return dialog.get()
 
diff --git a/lib/matplotlib/backends/qt_editor/figureoptions.py b/lib/matplotlib/backends/qt_editor/figureoptions.py
@@ -255,6 +255,4 @@ def apply_callback(data):
         datalist, title="Figure options", parent=parent,
         icon=QtGui.QIcon(
             str(cbook._get_data_path('images', 'qt4_editor_options.svg'))),
-        apply=apply_callback)
-    if data is not None:
-        apply_callback(data)
+        apply=apply_callback, modal=False)
