openlp-core team mailing list archive

[Jonathan Springer <springermac@xxxxxxxxx>]

Jonathan Springer has proposed merging lp:~springermac/openlp/mac_dock_fix into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #1247661 in OpenLP: "Cannot restore OpenLP after minimising on OSX"
  https://bugs.launchpad.net/openlp/+bug/1247661

For more details, see:
https://code.launchpad.net/~springermac/openlp/mac_dock_fix/+merge/257435

Fix bug 1247661 by restoring to main window when the dock icon is clicked
-- 
Your team OpenLP Core is requested to review the proposed merge of lp:~springermac/openlp/mac_dock_fix into lp:openlp.

=== modified file 'openlp/core/__init__.py'
--- openlp/core/__init__.py	2015-01-22 20:12:23 +0000
+++ openlp/core/__init__.py	2015-04-24 20:27:04 +0000
@@ -250,7 +250,7 @@
 
     def event(self, event):
         """
-        Enables direct file opening on OS X
+        Enables platform specific event handling i.e. direct file opening on OS X
 
         :param event: The event
         """
@@ -259,8 +259,19 @@
             log.debug('Got open file event for %s!', file_name)
             self.args.insert(0, file_name)
             return True
-        else:
-            return QtGui.QApplication.event(self, event)
+        # Mac OS X should restore app window when user clicked on the OpenLP icon
+        # in the Dock bar. However, OpenLP consists of multiple windows and this
+        # does not work. This workaround fixes that.
+        # The main OpenLP window is restored when it was previously minimized.
+        elif event.type() == QtCore.QEvent.ApplicationActivate:
+            if is_macosx() and hasattr(self, 'main_window'):
+                if self.main_window.isMinimized():
+                    # Copied from QWidget.setWindowState() docs on how to restore and activate a minimized window
+                    # while preserving its maximized and/or full-screen state.
+                    self.main_window.setWindowState(self.main_window.windowState() & ~QtCore.Qt.WindowMinimized |
+                                                    QtCore.Qt.WindowActive)
+                    return True
+        return QtGui.QApplication.event(self, event)
 
 
 def parse_options(args):