- Removed 'Default' button

- Translate the tag description (bug #787552).

bzr-revno: 1577
Fixes: https://launchpad.net/bugs/787552

openlp/core/lib/__init__.py

@@ -36,58 +36,6 @@ from PyQt4 import QtCore, QtGui
 
 log = logging.getLogger(__name__)
 
-base_html_expands = []
-
-# Hex Color tags from http://www.w3schools.com/html/html_colornames.asp
-base_html_expands.append({u'desc': u'Red', u'start tag': u'{r}',
-    u'start html': u'<span style="-webkit-text-fill-color:red">',
-    u'end tag': u'{/r}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Black', u'start tag': u'{b}',
-    u'start html': u'<span style="-webkit-text-fill-color:black">',
-    u'end tag': u'{/b}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Blue', u'start tag': u'{bl}',
-    u'start html': u'<span style="-webkit-text-fill-color:blue">',
-    u'end tag': u'{/bl}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Yellow', u'start tag': u'{y}',
-    u'start html': u'<span style="-webkit-text-fill-color:yellow">',
-    u'end tag': u'{/y}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Green', u'start tag': u'{g}',
-    u'start html': u'<span style="-webkit-text-fill-color:green">',
-    u'end tag': u'{/g}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Pink', u'start tag': u'{pk}',
-    u'start html': u'<span style="-webkit-text-fill-color:#FFC0CB">',
-    u'end tag': u'{/pk}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Orange', u'start tag': u'{o}',
-    u'start html': u'<span style="-webkit-text-fill-color:#FFA500">',
-    u'end tag': u'{/o}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Purple', u'start tag': u'{pp}',
-    u'start html': u'<span style="-webkit-text-fill-color:#800080">',
-    u'end tag': u'{/pp}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'White', u'start tag': u'{w}',
-    u'start html': u'<span style="-webkit-text-fill-color:white">',
-    u'end tag': u'{/w}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Superscript', u'start tag': u'{su}',
-    u'start html': u'<sup>', u'end tag': u'{/su}', u'end html': u'</sup>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Subscript', u'start tag': u'{sb}',
-    u'start html': u'<sub>', u'end tag': u'{/sb}', u'end html': u'</sub>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Paragraph', u'start tag': u'{p}',
-    u'start html': u'<p>', u'end tag': u'{/p}', u'end html': u'</p>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Bold', u'start tag': u'{st}',
-    u'start html': u'<strong>', u'end tag': u'{/st}', u'end html': u'</strong>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Italics', u'start tag': u'{it}',
-    u'start html': u'<em>', u'end tag': u'{/it}', u'end html': u'</em>',
-    u'protected': True})
-base_html_expands.append({u'desc': u'Underline', u'start tag': u'{u}',
-    u'start html': u'<span style="text-decoration: underline;">',
-    u'end tag': u'{/u}', u'end html': u'</span>', u'protected': True})
-base_html_expands.append({u'desc': u'Break', u'start tag': u'{br}',
-    u'start html': u'<br>', u'end tag': u'', u'end html': u'',
-    u'protected': True})
-
 def translate(context, text, comment=None,
     encoding=QtCore.QCoreApplication.CodecForTr, n=-1,
     translate=QtCore.QCoreApplication.translate):

openlp/core/lib/displaytags.py

@@ -28,7 +28,7 @@
 Provide Html Tag management and Display Tag access class
 """
 
-from openlp.core.lib import base_html_expands
+from openlp.core.lib import translate
 
 class DisplayTags(object):
     """
@@ -50,15 +50,77 @@ class DisplayTags(object):
         Resets the html_expands list.
         """
         DisplayTags.html_expands = []
-        for html in base_html_expands:
+        base_tags = []
-            DisplayTags.html_expands.append(html)
+        # Append the base tags.
+        # Hex Color tags from http://www.w3schools.com/html/html_colornames.asp
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Red'),
+            u'start tag': u'{r}',
+            u'start html': u'<span style="-webkit-text-fill-color:red">',
+            u'end tag': u'{/r}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc':  translate('OpenLP.DisplayTags', 'Black'),
+            u'start tag': u'{b}',
+            u'start html': u'<span style="-webkit-text-fill-color:black">',
+            u'end tag': u'{/b}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Blue'),
+            u'start tag': u'{bl}',
+            u'start html': u'<span style="-webkit-text-fill-color:blue">',
+            u'end tag': u'{/bl}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Yellow'),
+            u'start tag': u'{y}',
+            u'start html': u'<span style="-webkit-text-fill-color:yellow">',
+            u'end tag': u'{/y}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Green'),
+            u'start tag': u'{g}',
+            u'start html': u'<span style="-webkit-text-fill-color:green">',
+            u'end tag': u'{/g}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Pink'),
+            u'start tag': u'{pk}',
+            u'start html': u'<span style="-webkit-text-fill-color:#FFC0CB">',
+            u'end tag': u'{/pk}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Orange'),
+            u'start tag': u'{o}',
+            u'start html': u'<span style="-webkit-text-fill-color:#FFA500">',
+            u'end tag': u'{/o}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Purple'),
+            u'start tag': u'{pp}',
+            u'start html': u'<span style="-webkit-text-fill-color:#800080">',
+            u'end tag': u'{/pp}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'White'),
+            u'start tag': u'{w}',
+            u'start html': u'<span style="-webkit-text-fill-color:white">',
+            u'end tag': u'{/w}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({
+            u'desc': translate('OpenLP.DisplayTags', 'Superscript'),
+            u'start tag': u'{su}', u'start html': u'<sup>',
+            u'end tag': u'{/su}', u'end html': u'</sup>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Subscript'),
+            u'start tag': u'{sb}', u'start html': u'<sub>',
+            u'end tag': u'{/sb}', u'end html': u'</sub>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Paragraph'),
+            u'start tag': u'{p}', u'start html': u'<p>', u'end tag': u'{/p}',
+            u'end html': u'</p>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Bold'),
+            u'start tag': u'{st}', u'start html': u'<strong>',
+            u'end tag': u'{/st}', u'end html': u'</strong>',
+            u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Italics'),
+            u'start tag': u'{it}', u'start html': u'<em>', u'end tag': u'{/it}',
+            u'end html': u'</em>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Underline'),
+            u'start tag': u'{u}',
+            u'start html': u'<span style="text-decoration: underline;">',
+            u'end tag': u'{/u}', u'end html': u'</span>', u'protected': True})
+        base_tags.append({u'desc': translate('OpenLP.DisplayTags', 'Break'),
+            u'start tag': u'{br}', u'start html': u'<br>', u'end tag': u'',
+            u'end html': u'', u'protected': True})
+        DisplayTags.add_html_tags(base_tags)
 
     @staticmethod
-    def add_html_tag(tag):
+    def add_html_tags(tags):
         """
-        Add a new tag to the list
+        Add a list of tags to the list
         """
-        DisplayTags.html_expands.append(tag)
+        DisplayTags.html_expands.extend(tags)
 
     @staticmethod
     def remove_html_tag(tag_id):

openlp/core/ui/displaytagdialog.py

@@ -69,9 +69,6 @@ class Ui_DisplayTagDialog(object):
         spacerItem = QtGui.QSpacerItem(40, 20,
             QtGui.QSizePolicy.Expanding, QtGui.QSizePolicy.Minimum)
         self.horizontalLayout.addItem(spacerItem)
-        self.defaultPushButton = QtGui.QPushButton(self.widget)
-        self.defaultPushButton.setObjectName(u'defaultPushButton')
-        self.horizontalLayout.addWidget(self.defaultPushButton)
         self.deletePushButton = QtGui.QPushButton(self.widget)
         self.deletePushButton.setObjectName(u'deletePushButton')
         self.horizontalLayout.addWidget(self.deletePushButton)
@@ -141,8 +138,6 @@ class Ui_DisplayTagDialog(object):
         self.endTagLabel.setText(
             translate('OpenLP.DisplayTagDialog', 'End tag'))
         self.deletePushButton.setText(UiStrings().Delete)
-        self.defaultPushButton.setText(
-            translate('OpenLP.DisplayTagDialog', 'Default'))
         self.newPushButton.setText(UiStrings().New)
         self.tagTableWidget.horizontalHeaderItem(0).setText(
             translate('OpenLP.DisplayTagDialog', 'Description'))

openlp/core/ui/displaytagform.py

@@ -51,8 +51,6 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
         self._loadDisplayTags()
         QtCore.QObject.connect(self.tagTableWidget,
             QtCore.SIGNAL(u'clicked(QModelIndex)'), self.onRowSelected)
-        QtCore.QObject.connect(self.defaultPushButton,
-            QtCore.SIGNAL(u'pressed()'), self.onDefaultPushed)
         QtCore.QObject.connect(self.newPushButton,
             QtCore.SIGNAL(u'pressed()'), self.onNewPushed)
         QtCore.QObject.connect(self.savePushButton,
@@ -87,8 +85,7 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
         if user_expands_string:
             user_tags = cPickle.loads(user_expands_string)
             # If we have some user ones added them as well
-            for t in user_tags:
+            DisplayTags.add_html_tags(user_tags)
-                DisplayTags.add_html_tag(t)
 
     def onRowSelected(self):
         """
@@ -128,22 +125,20 @@ class DisplayTagForm(QtGui.QDialog, Ui_DisplayTagDialog):
                     'Tag "n" already defined.'))
                 return
         # Add new tag to list
-        tag = {u'desc': u'New Item', u'start tag': u'{n}',
+        tag = {
-            u'start html': u'<Html_here>', u'end tag': u'{/n}',
+            u'desc': translate('OpenLP.DisplayTagTab', 'New Tag'),
-            u'end html': u'</and here>', u'protected': False}
+            u'start tag': u'{n}',
-        DisplayTags.add_html_tag(tag)
+            u'start html': translate('OpenLP.DisplayTagTab', '<Html_here>'),
+            u'end tag': u'{/n}',
+            u'end html': translate('OpenLP.DisplayTagTab', '</and here>'),
+            u'protected': False
+        }
+        DisplayTags.add_html_tags([tag])
         self._resetTable()
         # Highlight new row
         self.tagTableWidget.selectRow(self.tagTableWidget.rowCount() - 1)
         self.onRowSelected()
 
-    def onDefaultPushed(self):
-        """
-        Remove all Custom Tags and reset to base set only.
-        """
-        DisplayTags.reset_html_tags()
-        self._resetTable()
-
     def onDeletePushed(self):
         """
         Delete selected custom tag.