Make Windows the default window style on windows, not GTK+

Author: Lee Miller

src/bitmessageqt/__init__.py

@@ -62,6 +62,9 @@
     get_plugins = False
 
 
+is_windows = sys.platform.startswith('win')
+
+
 # TODO: rewrite
 def powQueueSize():
     """Returns the size of queues.workerQueue including current unfinished work"""
@@ -80,7 +83,7 @@ def openKeysFile():
     keysfile = os.path.join(state.appdata, 'keys.dat')
     if 'linux' in sys.platform:
         subprocess.call(["xdg-open", keysfile])
-    elif sys.platform.startswith('win'):
+    elif is_windows:
         os.startfile(keysfile)  # pylint: disable=no-member
 
 
@@ -868,7 +871,7 @@ def updateStartOnLogon(self):
         """
         startonlogon = config.safeGetBoolean(
             'bitmessagesettings', 'startonlogon')
-        if sys.platform.startswith('win'):  # Auto-startup for Windows
+        if is_windows:  # Auto-startup for Windows
             RUN_PATH = "HKEY_CURRENT_USER\\Software\\Microsoft\\Windows\\CurrentVersion\\Run"
             settings = QtCore.QSettings(
                 RUN_PATH, QtCore.QSettings.NativeFormat)
@@ -4233,6 +4236,14 @@ class BitmessageQtApplication(QtGui.QApplication):
     # Unique identifier for this application
     uuid = '6ec0149b-96e1-4be1-93ab-1465fb3ebf7c'
 
+    @staticmethod
+    def get_windowstyle():
+        """Get window style set in config or default"""
+        return config.safeGet(
+            'bitmessagesettings', 'windowstyle',
+            'Windows' if is_windows else 'GTK+'
+        )
+
     def __init__(self, *argv):
         super(BitmessageQtApplication, self).__init__(*argv)
         id = BitmessageQtApplication.uuid
@@ -4241,8 +4252,7 @@ def __init__(self, *argv):
         QtCore.QCoreApplication.setOrganizationDomain("bitmessage.org")
         QtCore.QCoreApplication.setApplicationName("pybitmessageqt")
 
-        self.setStyle(
-            config.safeGet('bitmessagesettings', 'windowstyle', 'GTK+'))
+        self.setStyle(self.get_windowstyle())
 
         font = config.safeGet('bitmessagesettings', 'font')
         if font:

src/bitmessageqt/settings.py

@@ -45,6 +45,7 @@ def __init__(self, parent=None, firstrun=False):
         super(SettingsDialog, self).__init__(parent)
         widgets.load('settings.ui', self)
 
+        self.app = QtGui.QApplication.instance()
         self.parent = parent
         self.firstrun = firstrun
         self.config = config_obj
@@ -87,14 +88,13 @@ def adjust_from_config(self, config):
         """Adjust all widgets state according to config settings"""
         # pylint: disable=too-many-branches,too-many-statements
 
-        current_style = config.safeGet(
-            'bitmessagesettings', 'windowstyle', 'GTK+')
+        current_style = self.app.get_windowstyle()
         for i, sk in enumerate(QtGui.QStyleFactory.keys()):
             self.comboBoxStyle.addItem(sk)
             if sk == current_style:
                 self.comboBoxStyle.setCurrentIndex(i)
 
-        self.save_font_setting(QtGui.QApplication.instance().font())
+        self.save_font_setting(self.app.font())
 
         if not self.parent.tray.isSystemTrayAvailable():
             self.groupBoxTray.setEnabled(False)
@@ -148,7 +148,7 @@ def adjust_from_config(self, config):
                 "MainWindow",
                 "Tray notifications not yet supported on your OS."))
 
-        if 'win' not in sys.platform and not self.parent.desktop:
+        if not sys.platform.startswith('win') and not self.parent.desktop:
             self.checkBoxStartOnLogon.setDisabled(True)
             self.checkBoxStartOnLogon.setText(_translate(
                 "MainWindow", "Start-on-login not yet supported on your OS."))
@@ -372,9 +372,7 @@ def accept(self):
             self.checkBoxReplyBelow.isChecked()))
 
         window_style = str(self.comboBoxStyle.currentText())
-        if self.config.safeGet(
-            'bitmessagesettings', 'windowstyle', 'GTK+'
-        ) != window_style or self.config.safeGet(
+        if self.app.get_windowstyle() != window_style or self.config.safeGet(
             'bitmessagesettings', 'font'
         ) != self.font_setting:
             self.config.set('bitmessagesettings', 'windowstyle', window_style)

src/bitmessageqt/tests/settings.py

@@ -43,7 +43,8 @@ def test_styling(self):
         self.assertIs(style_setting, None)
         self.assertIs(font_setting, None)
         style_control = self.dialog.comboBoxStyle
-        self.assertEqual(style_control.currentText(), 'GTK+')
+        self.assertEqual(
+            style_control.currentText(), self.app.get_windowstyle())
 
         def call_font_dialog():
             """A function to get the open font dialog and accept it"""