From c9c126b67f1a13483075aae680b30813117fbb05 Mon Sep 17 00:00:00 2001
From: Kali Kaneko <kali@leap.se>
Date: Tue, 9 Jul 2013 02:27:36 +0900
Subject: do not use exec_ on wizard Closes: #3047

---
 src/leap/gui/mainwindow.py | 18 ++++++++++++++----
 1 file changed, 14 insertions(+), 4 deletions(-)

(limited to 'src/leap/gui/mainwindow.py')

diff --git a/src/leap/gui/mainwindow.py b/src/leap/gui/mainwindow.py
index 7180139a..52caf08e 100644
--- a/src/leap/gui/mainwindow.py
+++ b/src/leap/gui/mainwindow.py
@@ -310,12 +310,22 @@ class MainWindow(QtGui.QMainWindow):
         if self._wizard is None:
             self._wizard = Wizard(bypass_checks=self._bypass_checks)
             self._wizard.accepted.connect(self._finish_init)
+            self._wizard.rejected.connect(self._wizard.close)
 
         self.setVisible(False)
-        self._wizard.exec_()
-        # We need this to process any wizard related event
-        QtCore.QCoreApplication.processEvents()
-        self._wizard = None
+        # Do NOT use exec_, it will use a child event loop!
+        # Refer to http://www.themacaque.com/?p=1067 for funny details.
+        self._wizard.show()
+        self._wizard.finished.connect(self._wizard_finished)
+
+    def _wizard_finished(self):
+        """
+        SLOT
+        TRIGGERS
+          self._wizard.finished
+
+        Called when the wizard has finished.
+        """
         self.setVisible(True)
 
     def _get_leap_logging_handler(self):
-- 
cgit v1.2.3