implement live syntax checking for rename strings

picard/ui/options/renaming.py

@@ -36,6 +36,8 @@ class RenamingOptionsPage(OptionsPage):
     SORT_ORDER = 40
     ACTIVE = True
 
+    STYLESHEET_ERROR = "QWidget { background-color: #400; color: white; font-weight:bold }"
+    
     options = [
         BoolOption("setting", "windows_compatible_filenames", True),
         BoolOption("setting", "ascii_filenames", False),
@@ -106,11 +108,15 @@ class RenamingOptionsPage(OptionsPage):
         self.ui.va_file_naming_format.setText(self.ui.file_naming_format.toPlainText())
 
     def test(self):
+        self.ui.renaming_error.setStyleSheet("");
+        self.ui.renaming_error.setText("")
         try:
             self.check()
         except OptionsCheckError, e:
-            dialog = QtGui.QMessageBox(QtGui.QMessageBox.Warning, e.title, e.message, QtGui.QMessageBox.Ok, self)
-            dialog.exec_()
+            self.ui.renaming_error.setStyleSheet(self.STYLESHEET_ERROR);
+            self.ui.renaming_error.setText(e.message)
+#            dialog = QtGui.QMessageBox(QtGui.QMessageBox.Warning, e.title, e.message, QtGui.QMessageBox.Ok, self)
+#            dialog.exec_()
             return
         
         settings = {
@@ -153,12 +159,16 @@ class RenamingOptionsPage(OptionsPage):
         self.ui.example_filename.setText(filename)
 
     def va_test(self):
+        self.ui.renaming_error.setStyleSheet("");
+        self.ui.renaming_error.setText("")
 
         try:
             self.check()
         except OptionsCheckError, e:
-            dialog = QtGui.QMessageBox(QtGui.QMessageBox.Warning, e.title, e.message, QtGui.QMessageBox.Ok, self)
-            dialog.exec_()
+            self.ui.renaming_error.setStyleSheet(self.STYLESHEET_ERROR);
+            self.ui.renaming_error.setText(e.message)
+#            dialog = QtGui.QMessageBox(QtGui.QMessageBox.Warning, e.title, e.message, QtGui.QMessageBox.Ok, self)
+#            dialog.exec_()
             return
         
         settings = {

picard/ui/ui_options_renaming.py

@@ -2,7 +2,7 @@
 
 # Form implementation generated from reading ui file 'ui/options_renaming.ui'
 #
-# Created: Sun Jan  4 22:14:10 2009
+# Created: Sun Jan  4 22:53:33 2009
 #      by: PyQt4 UI code generator 4.4.3
 #
 # WARNING! All changes made in this file will be lost!
@@ -13,6 +13,11 @@ class Ui_RenamingOptionsPage(object):
     def setupUi(self, RenamingOptionsPage):
         RenamingOptionsPage.setObjectName("RenamingOptionsPage")
         RenamingOptionsPage.resize(892, 782)
+        sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Expanding, QtGui.QSizePolicy.Preferred)
+        sizePolicy.setHorizontalStretch(0)
+        sizePolicy.setVerticalStretch(0)
+        sizePolicy.setHeightForWidth(RenamingOptionsPage.sizePolicy().hasHeightForWidth())
+        RenamingOptionsPage.setSizePolicy(sizePolicy)
         self.vboxlayout = QtGui.QVBoxLayout(RenamingOptionsPage)
         self.vboxlayout.setObjectName("vboxlayout")
         self.horizontalLayout = QtGui.QHBoxLayout()
@@ -58,6 +63,11 @@ class Ui_RenamingOptionsPage(object):
         self.horizontalLayout_2 = QtGui.QHBoxLayout()
         self.horizontalLayout_2.setObjectName("horizontalLayout_2")
         self.label_3 = QtGui.QLabel(self.groupBox)
+        sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Fixed, QtGui.QSizePolicy.Preferred)
+        sizePolicy.setHorizontalStretch(0)
+        sizePolicy.setVerticalStretch(0)
+        sizePolicy.setHeightForWidth(self.label_3.sizePolicy().hasHeightForWidth())
+        self.label_3.setSizePolicy(sizePolicy)
         font = QtGui.QFont()
         font.setWeight(75)
         font.setBold(True)
@@ -65,6 +75,10 @@ class Ui_RenamingOptionsPage(object):
         self.label_3.setAlignment(QtCore.Qt.AlignLeading|QtCore.Qt.AlignLeft|QtCore.Qt.AlignVCenter)
         self.label_3.setObjectName("label_3")
         self.horizontalLayout_2.addWidget(self.label_3)
+        self.renaming_error = QtGui.QLabel(self.groupBox)
+        self.renaming_error.setAlignment(QtCore.Qt.AlignCenter)
+        self.renaming_error.setObjectName("renaming_error")
+        self.horizontalLayout_2.addWidget(self.renaming_error)
         self.file_naming_format_default = QtGui.QPushButton(self.groupBox)
         sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Fixed, QtGui.QSizePolicy.Fixed)
         sizePolicy.setHorizontalStretch(0)
@@ -115,6 +129,11 @@ class Ui_RenamingOptionsPage(object):
         self.horizontalLayout_3 = QtGui.QHBoxLayout()
         self.horizontalLayout_3.setObjectName("horizontalLayout_3")
         self.label_4 = QtGui.QLabel(self.use_va_format)
+        sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Fixed, QtGui.QSizePolicy.Preferred)
+        sizePolicy.setHorizontalStretch(0)
+        sizePolicy.setVerticalStretch(0)
+        sizePolicy.setHeightForWidth(self.label_4.sizePolicy().hasHeightForWidth())
+        self.label_4.setSizePolicy(sizePolicy)
         font = QtGui.QFont()
         font.setWeight(75)
         font.setBold(True)
@@ -122,6 +141,10 @@ class Ui_RenamingOptionsPage(object):
         self.label_4.setAlignment(QtCore.Qt.AlignLeading|QtCore.Qt.AlignLeft|QtCore.Qt.AlignVCenter)
         self.label_4.setObjectName("label_4")
         self.horizontalLayout_3.addWidget(self.label_4)
+        self.renaming_va_error = QtGui.QLabel(self.use_va_format)
+        self.renaming_va_error.setAlignment(QtCore.Qt.AlignCenter)
+        self.renaming_va_error.setObjectName("renaming_va_error")
+        self.horizontalLayout_3.addWidget(self.renaming_va_error)
         self.va_copy_from_above = QtGui.QPushButton(self.use_va_format)
         sizePolicy = QtGui.QSizePolicy(QtGui.QSizePolicy.Fixed, QtGui.QSizePolicy.Fixed)
         sizePolicy.setHorizontalStretch(0)

ui/options_renaming.ui

@@ -9,6 +9,12 @@
     <height>782</height>
    </rect>
   </property>
+  <property name="sizePolicy" >
+   <sizepolicy vsizetype="Preferred" hsizetype="Expanding" >
+    <horstretch>0</horstretch>
+    <verstretch>0</verstretch>
+   </sizepolicy>
+  </property>
   <property name="windowTitle" >
    <string>Form</string>
   </property>
@@ -87,6 +93,12 @@
        <layout class="QHBoxLayout" name="horizontalLayout_2" >
         <item>
          <widget class="QLabel" name="label_3" >
+          <property name="sizePolicy" >
+           <sizepolicy vsizetype="Preferred" hsizetype="Fixed" >
+            <horstretch>0</horstretch>
+            <verstretch>0</verstretch>
+           </sizepolicy>
+          </property>
           <property name="font" >
            <font>
             <weight>75</weight>
@@ -101,6 +113,16 @@
           </property>
          </widget>
         </item>
+        <item>
+         <widget class="QLabel" name="renaming_error" >
+          <property name="text" >
+           <string/>
+          </property>
+          <property name="alignment" >
+           <set>Qt::AlignCenter</set>
+          </property>
+         </widget>
+        </item>
         <item>
          <widget class="QPushButton" name="file_naming_format_default" >
           <property name="sizePolicy" >
@@ -211,6 +233,12 @@ p, li { white-space: pre-wrap; }
        <layout class="QHBoxLayout" name="horizontalLayout_3" >
         <item>
          <widget class="QLabel" name="label_4" >
+          <property name="sizePolicy" >
+           <sizepolicy vsizetype="Preferred" hsizetype="Fixed" >
+            <horstretch>0</horstretch>
+            <verstretch>0</verstretch>
+           </sizepolicy>
+          </property>
           <property name="font" >
            <font>
             <weight>75</weight>
@@ -225,6 +253,19 @@ p, li { white-space: pre-wrap; }
           </property>
          </widget>
         </item>
+        <item>
+         <widget class="QLabel" name="renaming_va_error" >
+          <property name="styleSheet" >
+           <string notr="true" />
+          </property>
+          <property name="text" >
+           <string/>
+          </property>
+          <property name="alignment" >
+           <set>Qt::AlignCenter</set>
+          </property>
+         </widget>
+        </item>
         <item>
          <widget class="QPushButton" name="va_copy_from_above" >
           <property name="sizePolicy" >