← Back to team overview

openlp-core team mailing list archive

[Merge] lp:~raoul-snyman/openlp/bug-752212 into lp:openlp

 

Raoul Snyman has proposed merging lp:~raoul-snyman/openlp/bug-752212 into lp:openlp.

Requested reviews:
  OpenLP Core (openlp-core)
Related bugs:
  Bug #752212 in OpenLP: "Initial screen background reverts to white"
  https://bugs.launchpad.net/openlp/+bug/752212

For more details, see:
https://code.launchpad.net/~raoul-snyman/openlp/bug-752212/+merge/59185

Fixed bug #752212. Added a revert button to revert the image to the default OpenLP image. Also moved things slightly to for the dialog to look a little better on other platforms/DEs.
-- 
https://code.launchpad.net/~raoul-snyman/openlp/bug-752212/+merge/59185
Your team OpenLP Core is requested to review the proposed merge of lp:~raoul-snyman/openlp/bug-752212 into lp:openlp.
=== modified file 'openlp/core/resources.py'
--- openlp/core/resources.py	2011-03-24 19:04:02 +0000
+++ openlp/core/resources.py	2011-04-27 09:03:53 +0000
@@ -81470,6 +81470,55 @@
 \x48\x78\x87\x12\xd5\x64\x32\xf9\x9d\xaa\xe7\x00\xd4\x2d\xf8\x3b\
 \x24\xa2\x8d\xd8\x23\x1a\x44\x8e\x7f\x81\xdf\xa3\x70\x33\x35\x18\
 \x40\x99\x79\x00\x00\x00\x00\x49\x45\x4e\x44\xae\x42\x60\x82\
+\x00\x00\x02\xe1\
+\x89\
+\x50\x4e\x47\x0d\x0a\x1a\x0a\x00\x00\x00\x0d\x49\x48\x44\x52\x00\
+\x00\x00\x10\x00\x00\x00\x10\x08\x06\x00\x00\x00\x1f\xf3\xff\x61\
+\x00\x00\x02\xa8\x49\x44\x41\x54\x78\xda\x85\x93\x5f\x48\x53\x51\
+\x1c\xc7\x17\x24\xc5\x0a\x91\x84\xfe\x58\xad\xa7\x0c\x8a\x90\x10\
+\xa3\x29\x65\x3e\x14\xd9\x93\x06\x9a\x86\x11\x65\xcd\x69\x43\x2b\
+\x85\x5a\x56\x0f\xb9\x34\x34\xd7\x28\x87\xf9\x50\x09\x81\x81\xe0\
+\x83\x92\xe6\x60\x56\x8e\x4c\x1f\x54\x6a\x9a\xe1\xb4\x6c\xa1\xb9\
+\x7f\x6e\xbb\xbb\x77\x77\x6e\xbb\xbb\xdf\xce\xee\xe6\xd4\x30\x3a\
+\xf0\xe1\xf0\x83\xf3\xfd\xfc\x7e\x70\xce\x11\x01\x10\x20\x2b\x81\
+\x20\x8d\x92\xfe\x7f\x84\x73\x09\x2b\x05\x52\x9b\xcd\x06\x8b\xd5\
+\x0a\x9b\xdd\x0e\xc7\xc2\x02\x16\x9c\x4e\x38\x5d\x2e\xb8\xdc\x6e\
+\xb8\x29\x4a\x80\xf2\x78\x04\x68\x86\x81\x20\x59\x21\x48\xb7\x58\
+\xff\x29\x88\x85\x3d\x51\x98\x88\x20\x7d\x95\xc0\xee\x70\x80\x10\
+\x0e\xc7\x82\x14\x45\x09\x01\x9a\xa6\x85\x10\xe3\xf5\xc2\x4b\x60\
+\x59\x36\x22\x50\xe6\xee\x4c\x5c\x12\x84\x3b\x8e\x0e\xbc\x45\x9b\
+\xe6\x2a\xea\xe4\xa9\xb8\x9d\x2f\x41\x83\x42\x0a\xdd\xeb\x7a\x98\
+\xa7\xbe\x80\xf5\xf9\xe0\x8b\xb2\xe8\xf7\x47\x04\x0f\xe5\x69\x46\
+\x65\x7e\x52\xd5\xfe\xdd\xe2\x9c\x47\x15\x47\xd1\xf9\x38\x03\xe6\
+\xa1\x22\x50\x33\xa5\x08\xb8\xaa\xe0\xf9\x29\x87\x79\xb0\x10\xba\
+\xe6\x2c\xbc\xac\x2d\x82\x8f\xf5\x22\x10\x08\x20\x18\x0c\x46\x04\
+\xcd\x95\x07\x38\x7d\xcb\x71\xbe\xe6\x62\x72\xc8\xa4\x2f\x84\xef\
+\x77\x19\xa1\x64\x0d\x4a\xf1\xfd\x43\x01\xd4\x8a\x43\xa0\x5c\xf6\
+\x65\x81\xf6\xda\x3e\x8e\x36\xc9\xe1\xfe\x56\x0a\x7a\xba\x44\x80\
+\x1a\x2b\x86\xa3\x3f\x17\xf3\x5d\x87\x61\x7f\x7f\x12\x9e\xf1\x4b\
+\x60\x4c\x57\x04\x66\x3f\x9e\x83\xba\xfc\xd8\xb2\xe0\x69\x79\x32\
+\xe7\x32\x16\xc3\x39\x4e\xf8\x5a\x8c\xd9\xbe\x0c\x4c\x3c\x4f\x40\
+\xcf\xbd\x78\xbc\x52\xc4\xe3\x4d\x95\x18\x93\xea\x4d\xb0\xea\x4e\
+\xc1\x3d\x72\x41\x40\xaf\xcd\x80\xa1\xeb\x59\x44\xf0\x44\xb1\x97\
+\xb3\x0d\x17\xc1\xfe\xf9\x3c\x7e\xf5\x4b\x31\xd1\x11\x8f\xc6\x92\
+\x1d\xe8\x6d\xab\xc7\x94\x71\x00\x1d\x2d\xb7\xa0\x29\x4b\xc1\xe8\
+\x03\x31\xac\xbd\xd9\x64\xb2\x7c\x58\xdf\xe5\xe1\x4e\xc1\xae\xc8\
+\x3b\x50\xcb\x24\xdc\x9c\x21\x1b\xf3\x23\x67\x30\x37\x92\x85\x99\
+\x21\x09\xda\x55\x89\x30\x0e\x76\x0b\x63\x72\x1c\x07\xda\xed\x40\
+\x7b\xcd\x09\x98\x5b\x0f\x12\x49\x0e\x2c\x3d\xa7\xd1\x5c\x91\x12\
+\x16\xa4\x89\xee\x9e\xdd\x6e\x69\xbc\xbc\x95\xd7\x94\x25\xf1\x4d\
+\xd7\xf7\x40\x5b\x29\x41\xd3\x8d\x6d\xd0\x2a\x33\xe1\x71\xd9\x11\
+\x0a\x85\xc0\xf3\x3c\xfa\xda\x1b\x31\xdc\x9a\x87\x81\x2e\x0d\x8c\
+\x43\xdd\x68\x50\xdd\x1c\x23\x82\x38\x82\x68\x9d\x4c\x96\x1a\xb7\
+\x79\xe3\xfa\x4c\xca\xed\x04\x43\x53\xb1\x6b\x22\xdd\x63\x82\x4f\
+\x3d\x2f\x60\xe8\xd4\x0a\x75\x43\x5d\xf5\x34\xc9\x89\x01\x88\x56\
+\xfd\x05\x96\xf5\x82\xf5\x7a\xe0\xf7\xfb\x57\x09\x16\x7d\x0c\xc6\
+\x0c\x6d\x18\x1d\xd4\xa3\x56\x55\x6d\x22\x67\xb7\x08\xb9\xbf\x04\
+\x47\x58\x96\xe1\x17\x7d\x34\x99\xc0\x1f\x46\x10\x04\xc9\xee\xb4\
+\xfc\x80\x65\x52\xc7\xab\xee\x2b\x87\x97\x3a\xaf\x25\xd8\x10\x96\
+\xac\xfd\xa5\x85\x3a\x8d\x10\x17\x0b\x47\xf9\x03\x3a\xca\x9b\x53\
+\x60\x80\xcc\x4d\x00\x00\x00\x00\x49\x45\x4e\x44\xae\x42\x60\x82\
+\
 \x00\x00\x02\x43\
 \x89\
 \x50\x4e\x47\x0d\x0a\x1a\x0a\x00\x00\x00\x0d\x49\x48\x44\x52\x00\
@@ -82083,6 +82132,11 @@
 \x09\xdf\xea\x07\
 \x00\x67\
 \x00\x65\x00\x6e\x00\x65\x00\x72\x00\x61\x00\x6c\x00\x5f\x00\x6c\x00\x69\x00\x76\x00\x65\x00\x2e\x00\x70\x00\x6e\x00\x67\
+\x00\x12\
+\x07\x2b\x7c\xe7\
+\x00\x67\
+\x00\x65\x00\x6e\x00\x65\x00\x72\x00\x61\x00\x6c\x00\x5f\x00\x72\x00\x65\x00\x76\x00\x65\x00\x72\x00\x74\x00\x2e\x00\x70\x00\x6e\
+\x00\x67\
 \x00\x0f\
 \x0d\x5b\xec\x87\
 \x00\x67\
@@ -82091,21 +82145,21 @@
 
 qt_resource_struct = "\
 \x00\x00\x00\x00\x00\x02\x00\x00\x00\x10\x00\x00\x00\x01\
-\x00\x00\x00\x98\x00\x02\x00\x00\x00\x06\x00\x00\x00\x79\
-\x00\x00\x00\x3a\x00\x02\x00\x00\x00\x04\x00\x00\x00\x75\
-\x00\x00\x01\x06\x00\x02\x00\x00\x00\x03\x00\x00\x00\x72\
-\x00\x00\x00\x5e\x00\x02\x00\x00\x00\x14\x00\x00\x00\x5e\
-\x00\x00\x00\x4e\x00\x02\x00\x00\x00\x02\x00\x00\x00\x5c\
-\x00\x00\x00\xe2\x00\x02\x00\x00\x00\x02\x00\x00\x00\x5a\
-\x00\x00\x00\x84\x00\x02\x00\x00\x00\x09\x00\x00\x00\x51\
-\x00\x00\x00\x14\x00\x02\x00\x00\x00\x0b\x00\x00\x00\x46\
-\x00\x00\x00\xa6\x00\x02\x00\x00\x00\x03\x00\x00\x00\x43\
-\x00\x00\x00\xf4\x00\x02\x00\x00\x00\x0f\x00\x00\x00\x34\
-\x00\x00\x00\x6e\x00\x02\x00\x00\x00\x03\x00\x00\x00\x31\
-\x00\x00\x01\x16\x00\x02\x00\x00\x00\x03\x00\x00\x00\x2e\
-\x00\x00\x00\xcc\x00\x02\x00\x00\x00\x08\x00\x00\x00\x26\
-\x00\x00\x00\x26\x00\x02\x00\x00\x00\x01\x00\x00\x00\x25\
-\x00\x00\x00\x00\x00\x02\x00\x00\x00\x0f\x00\x00\x00\x16\
+\x00\x00\x00\x98\x00\x02\x00\x00\x00\x06\x00\x00\x00\x7a\
+\x00\x00\x00\x3a\x00\x02\x00\x00\x00\x04\x00\x00\x00\x76\
+\x00\x00\x01\x06\x00\x02\x00\x00\x00\x03\x00\x00\x00\x73\
+\x00\x00\x00\x5e\x00\x02\x00\x00\x00\x14\x00\x00\x00\x5f\
+\x00\x00\x00\x4e\x00\x02\x00\x00\x00\x02\x00\x00\x00\x5d\
+\x00\x00\x00\xe2\x00\x02\x00\x00\x00\x02\x00\x00\x00\x5b\
+\x00\x00\x00\x84\x00\x02\x00\x00\x00\x09\x00\x00\x00\x52\
+\x00\x00\x00\x14\x00\x02\x00\x00\x00\x0b\x00\x00\x00\x47\
+\x00\x00\x00\xa6\x00\x02\x00\x00\x00\x03\x00\x00\x00\x44\
+\x00\x00\x00\xf4\x00\x02\x00\x00\x00\x0f\x00\x00\x00\x35\
+\x00\x00\x00\x6e\x00\x02\x00\x00\x00\x03\x00\x00\x00\x32\
+\x00\x00\x01\x16\x00\x02\x00\x00\x00\x03\x00\x00\x00\x2f\
+\x00\x00\x00\xcc\x00\x02\x00\x00\x00\x08\x00\x00\x00\x27\
+\x00\x00\x00\x26\x00\x02\x00\x00\x00\x01\x00\x00\x00\x26\
+\x00\x00\x00\x00\x00\x02\x00\x00\x00\x10\x00\x00\x00\x16\
 \x00\x00\x00\xb8\x00\x02\x00\x00\x00\x05\x00\x00\x00\x11\
 \x00\x00\x07\x82\x00\x00\x00\x00\x00\x01\x00\x0d\xce\x84\
 \x00\x00\x06\xc0\x00\x00\x00\x00\x00\x01\x00\x03\x4a\xfc\
@@ -82118,12 +82172,13 @@
 \x00\x00\x13\x4c\x00\x00\x00\x00\x00\x01\x00\x13\xcd\x4f\
 \x00\x00\x12\xce\x00\x00\x00\x00\x00\x01\x00\x13\xc2\xe7\
 \x00\x00\x11\x56\x00\x00\x00\x00\x00\x01\x00\x13\xa6\xf3\
+\x00\x00\x13\x9e\x00\x00\x00\x00\x00\x01\x00\x13\xd3\x56\
 \x00\x00\x12\x96\x00\x00\x00\x00\x00\x01\x00\x13\xbe\x45\
 \x00\x00\x11\x80\x00\x00\x00\x00\x00\x01\x00\x13\xa9\x91\
 \x00\x00\x13\x78\x00\x00\x00\x00\x00\x01\x00\x13\xd0\x82\
 \x00\x00\x13\x22\x00\x00\x00\x00\x00\x01\x00\x13\xca\xd7\
 \x00\x00\x11\xd4\x00\x00\x00\x00\x00\x01\x00\x13\xb1\x04\
-\x00\x00\x13\x9e\x00\x00\x00\x00\x00\x01\x00\x13\xd3\x56\
+\x00\x00\x13\xc8\x00\x00\x00\x00\x00\x01\x00\x13\xd6\x3b\
 \x00\x00\x12\xf6\x00\x00\x00\x00\x00\x01\x00\x13\xc6\x2c\
 \x00\x00\x12\x4a\x00\x00\x00\x00\x00\x01\x00\x13\xb7\xd4\
 \x00\x00\x12\x24\x00\x00\x00\x00\x00\x01\x00\x13\xb5\xbd\

=== modified file 'openlp/core/ui/advancedtab.py'
--- openlp/core/ui/advancedtab.py	2011-04-15 21:43:59 +0000
+++ openlp/core/ui/advancedtab.py	2011-04-27 09:03:53 +0000
@@ -41,11 +41,11 @@
         """
         Initialise the settings tab
         """
-        generalTranslated = translate('AdvancedTab', 'Advanced')
-        SettingsTab.__init__(self, parent ,u'Advanced', generalTranslated)
+        advancedTranslated = translate('OpenLP.AdvancedTab', 'Advanced')
         self.default_image = u':/graphics/openlp-splash-screen.png'
-        self.default_color = u'#ffffff'
+        self.default_color = u'#009900'
         self.icon_path = u':/system/system_settings.png'
+        SettingsTab.__init__(self, parent, u'Advanced', advancedTranslated)
 
     def setupUi(self):
         """
@@ -82,14 +82,6 @@
             u'enableAutoCloseCheckBox')
         self.uiLayout.addRow(self.enableAutoCloseCheckBox)
         self.leftLayout.addWidget(self.uiGroupBox)
-        self.hideMouseGroupBox = QtGui.QGroupBox(self.leftColumn)
-        self.hideMouseGroupBox.setObjectName(u'hideMouseGroupBox')
-        self.hideMouseLayout = QtGui.QVBoxLayout(self.hideMouseGroupBox)
-        self.hideMouseLayout.setObjectName(u'hideMouseLayout')
-        self.hideMouseCheckBox = QtGui.QCheckBox(self.hideMouseGroupBox)
-        self.hideMouseCheckBox.setObjectName(u'hideMouseCheckBox')
-        self.hideMouseLayout.addWidget(self.hideMouseCheckBox)
-        self.leftLayout.addWidget(self.hideMouseGroupBox)
         self.leftLayout.addStretch()
         self.defaultImageGroupBox = QtGui.QGroupBox(self.rightColumn)
         self.defaultImageGroupBox.setObjectName(u'defaultImageGroupBox')
@@ -109,26 +101,42 @@
         self.defaultBrowseButton.setObjectName(u'defaultBrowseButton')
         self.defaultBrowseButton.setIcon(
             build_icon(u':/general/general_open.png'))
+        self.defaultRevertButton = QtGui.QToolButton(self.defaultImageGroupBox)
+        self.defaultRevertButton.setObjectName(u'defaultRevertButton')
+        self.defaultRevertButton.setIcon(
+            build_icon(u':/general/general_revert.png'))
         self.defaultFileLayout = QtGui.QHBoxLayout()
         self.defaultFileLayout.setObjectName(u'defaultFileLayout')
         self.defaultFileLayout.addWidget(self.defaultFileEdit)
         self.defaultFileLayout.addWidget(self.defaultBrowseButton)
+        self.defaultFileLayout.addWidget(self.defaultRevertButton)
         self.defaultImageLayout.addRow(self.defaultFileLabel,
             self.defaultFileLayout)
         self.rightLayout.addWidget(self.defaultImageGroupBox)
+        self.hideMouseGroupBox = QtGui.QGroupBox(self.leftColumn)
+        self.hideMouseGroupBox.setObjectName(u'hideMouseGroupBox')
+        self.hideMouseLayout = QtGui.QVBoxLayout(self.hideMouseGroupBox)
+        self.hideMouseLayout.setObjectName(u'hideMouseLayout')
+        self.hideMouseCheckBox = QtGui.QCheckBox(self.hideMouseGroupBox)
+        self.hideMouseCheckBox.setObjectName(u'hideMouseCheckBox')
+        self.hideMouseLayout.addWidget(self.hideMouseCheckBox)
+        self.rightLayout.addWidget(self.hideMouseGroupBox)
         self.rightLayout.addStretch()
 
         QtCore.QObject.connect(self.defaultColorButton,
             QtCore.SIGNAL(u'pressed()'), self.onDefaultColorButtonPressed)
         QtCore.QObject.connect(self.defaultBrowseButton,
             QtCore.SIGNAL(u'pressed()'), self.onDefaultBrowseButtonPressed)
+        QtCore.QObject.connect(self.defaultRevertButton,
+            QtCore.SIGNAL(u'pressed()'), self.onDefaultRevertButtonPressed)
 
     def retranslateUi(self):
         """
         Setup the interface translation strings.
         """
         self.tabTitleVisible = UiStrings().Advanced
-        self.uiGroupBox.setTitle(translate('OpenLP.AdvancedTab', 'UI Settings'))
+        self.uiGroupBox.setTitle(
+            translate('OpenLP.AdvancedTab', 'UI Settings'))
         self.recentLabel.setText(
             translate('OpenLP.AdvancedTab',
                 'Number of recent files to display:'))
@@ -150,8 +158,14 @@
             'Default Image'))
         self.defaultColorLabel.setText(translate('OpenLP.AdvancedTab',
             'Background color:'))
+        self.defaultColorButton.setToolTip(translate('OpenLP.AdvancedTab',
+            'Click to select a color.'))
         self.defaultFileLabel.setText(translate('OpenLP.AdvancedTab',
             'Image file:'))
+        self.defaultBrowseButton.setToolTip(translate('OpenLP.AdvancedTab',
+            'Browse for an image file to display.'))
+        self.defaultRevertButton.setToolTip(translate('OpenLP.AdvancedTab',
+            'Revert to the default OpenLP logo.'))
 
     def load(self):
         """
@@ -184,7 +198,7 @@
         self.hideMouseCheckBox.setChecked(
             settings.value(u'hide mouse', QtCore.QVariant(False)).toBool())
         self.default_color = settings.value(u'default color',
-            QtCore.QVariant(u'#ffffff')).toString()
+            QtCore.QVariant(u'#000099')).toString()
         self.defaultFileEdit.setText(settings.value(u'default image',
             QtCore.QVariant(u':/graphics/openlp-splash-screen.png'))\
             .toString())
@@ -232,4 +246,8 @@
             file_filters)
         if filename:
             self.defaultFileEdit.setText(filename)
-        self.defaultFileEdit.setFocus()
\ No newline at end of file
+        self.defaultFileEdit.setFocus()
+
+    def onDefaultRevertButtonPressed(self):
+        self.defaultFileEdit.setText(u':/graphics/openlp-splash-screen.png')
+        self.defaultFileEdit.setFocus()

=== modified file 'openlp/core/ui/maindisplay.py'
--- openlp/core/ui/maindisplay.py	2011-04-22 07:06:38 +0000
+++ openlp/core/ui/maindisplay.py	2011-04-27 09:03:53 +0000
@@ -142,9 +142,10 @@
             image_file = QtCore.QSettings().value(u'advanced/default image',
                 QtCore.QVariant(u':/graphics/openlp-splash-screen.png'))\
                 .toString()
-            background_color = QtGui.QColor(QtCore.QSettings().value(
+            background_color = QtGui.QColor()
+            background_color.setNamedColor(QtCore.QSettings().value(
                 u'advanced/default color',
-                QtCore.QVariant(u'#ffffff')).toString())
+                QtCore.QVariant(u'#990000')).toString())
             if not background_color.isValid():
                 background_color = QtCore.Qt.white
             splash_image = QtGui.QImage(image_file)

=== added file 'resources/images/general_revert.png'
Binary files resources/images/general_revert.png	1970-01-01 00:00:00 +0000 and resources/images/general_revert.png	2011-04-27 09:03:53 +0000 differ
=== modified file 'resources/images/openlp-2.qrc'
--- resources/images/openlp-2.qrc	2011-03-09 16:16:40 +0000
+++ resources/images/openlp-2.qrc	2011-04-27 09:03:53 +0000
@@ -52,6 +52,7 @@
     <file>general_open.png</file>
     <file>general_save.png</file>
     <file>general_email.png</file>
+    <file>general_revert.png</file>
   </qresource>
   <qresource prefix="slides">
     <file>slide_close.png</file>


Follow ups