[QGIS Commit] r11743 - in trunk/qgis/src: app ui

[svn_qgis at osgeo.org]

Author: timlinux
Date: 2009-10-02 17:18:25 -0400 (Fri, 02 Oct 2009)
New Revision: 11743

Modified:
   trunk/qgis/src/app/qgsfieldcalculator.cpp
   trunk/qgis/src/ui/qgsfieldcalculatorbase.ui
Log:
Field Calculator: HIG compliance and use user friendly names for data types

Modified: trunk/qgis/src/app/qgsfieldcalculator.cpp
===================================================================
--- trunk/qgis/src/app/qgsfieldcalculator.cpp	2009-10-02 16:56:59 UTC (rev 11742)
+++ trunk/qgis/src/app/qgsfieldcalculator.cpp	2009-10-02 21:18:25 UTC (rev 11743)
@@ -22,9 +22,9 @@
 QgsFieldCalculator::QgsFieldCalculator( QgsVectorLayer* vl ): QDialog(), mVectorLayer( vl )
 {
   setupUi( this );
-  mOutputFieldTypeComboBox->addItem( tr( "Double" ) );
-  mOutputFieldTypeComboBox->addItem( tr( "Integer" ) );
-  mOutputFieldTypeComboBox->addItem( tr( "String" ) );
+  mOutputFieldTypeComboBox->addItem( tr( "Whole number (integer)" ), "Integer" );
+  mOutputFieldTypeComboBox->addItem( tr( "Decimal number (double)" ), "Double" );
+  mOutputFieldTypeComboBox->addItem( tr( "Text (string)" ), "String" );
 
   populateFields();
 
@@ -83,15 +83,16 @@
     {
       //create new field
       QgsField newField( mOutputFieldNameLineEdit->text() );
-      if ( mOutputFieldTypeComboBox->currentText() == tr( "Double" ) )
+      int index = mOutputFieldTypeComboBox->currentIndex();
+      if ( mOutputFieldTypeComboBox->itemData( index, Qt::UserRole ) ==  "Double" )
       {
         newField.setType( QVariant::Double );
       }
-      else if ( mOutputFieldTypeComboBox->currentText() == tr( "Integer" ) )
+      else if ( mOutputFieldTypeComboBox->itemData( index, Qt::UserRole ) == "Integer"  )
       {
         newField.setType( QVariant::Int );
       }
-      else if ( mOutputFieldTypeComboBox->currentText() == tr( "String" ) )
+      else if ( mOutputFieldTypeComboBox->itemData( index, Qt::UserRole ) == "String"  )
       {
         newField.setType( QVariant::String );
       }

Modified: trunk/qgis/src/ui/qgsfieldcalculatorbase.ui
===================================================================
--- trunk/qgis/src/ui/qgsfieldcalculatorbase.ui	2009-10-02 16:56:59 UTC (rev 11742)
+++ trunk/qgis/src/ui/qgsfieldcalculatorbase.ui	2009-10-02 21:18:25 UTC (rev 11743)
@@ -12,7 +12,7 @@
   <property name="windowTitle" >
    <string>Field calculator</string>
   </property>
-  <layout class="QGridLayout" name="gridLayout_5" >
+  <layout class="QGridLayout" >
    <item row="0" column="0" >
     <widget class="QCheckBox" name="mUpdateExistingFieldCheckBox" >
      <property name="text" >
@@ -35,12 +35,15 @@
      <property name="title" >
       <string>New field</string>
      </property>
-     <layout class="QGridLayout" name="gridLayout_3" >
+     <layout class="QGridLayout" >
       <item row="0" column="0" >
        <widget class="QLabel" name="mFieldNameLabel" >
         <property name="text" >
-         <string>Output field name:</string>
+         <string>Output field name</string>
         </property>
+        <property name="buddy" >
+         <cstring>mOutputFieldNameLineEdit</cstring>
+        </property>
        </widget>
       </item>
       <item row="0" column="1" colspan="3" >
@@ -49,8 +52,11 @@
       <item row="1" column="0" >
        <widget class="QLabel" name="mOutputFieldTypeLabel" >
         <property name="text" >
-         <string>Output field type:</string>
+         <string>Output field type</string>
         </property>
+        <property name="buddy" >
+         <cstring>mOutputFieldTypeComboBox</cstring>
+        </property>
        </widget>
       </item>
       <item row="1" column="1" colspan="3" >
@@ -59,8 +65,11 @@
       <item row="2" column="0" >
        <widget class="QLabel" name="mOutputFieldWidthLabel" >
         <property name="text" >
-         <string>Output field width:</string>
+         <string>Output field width</string>
         </property>
+        <property name="buddy" >
+         <cstring>mOuputFieldWidthSpinBox</cstring>
+        </property>
        </widget>
       </item>
       <item row="2" column="1" >
@@ -69,8 +78,11 @@
       <item row="2" column="2" >
        <widget class="QLabel" name="mOutputFieldPrecisionLabel" >
         <property name="text" >
-         <string>Output field precision:</string>
+         <string>Output field precision</string>
         </property>
+        <property name="buddy" >
+         <cstring>mOutputFieldPrecisionSpinBox</cstring>
+        </property>
        </widget>
       </item>
       <item row="2" column="3" >
@@ -84,7 +96,7 @@
      <property name="title" >
       <string>Fields</string>
      </property>
-     <layout class="QGridLayout" name="gridLayout_4" >
+     <layout class="QGridLayout" >
       <item row="0" column="0" >
        <widget class="QListWidget" name="mFieldsListWidget" />
       </item>
@@ -96,7 +108,7 @@
      <property name="title" >
       <string>Values</string>
      </property>
-     <layout class="QGridLayout" name="gridLayout_2" >
+     <layout class="QGridLayout" >
       <item row="0" column="0" colspan="2" >
        <widget class="QListWidget" name="mValueListWidget" />
       </item>
@@ -122,7 +134,7 @@
      <property name="title" >
       <string>Operators</string>
      </property>
-     <layout class="QGridLayout" name="gridLayout" >
+     <layout class="QGridLayout" >
       <item row="0" column="0" >
        <widget class="QPushButton" name="mPlusPushButton" >
         <property name="text" >
@@ -227,20 +239,23 @@
    <item row="5" column="0" colspan="2" >
     <widget class="QLabel" name="mFieldCalculatorExpressionLabel" >
      <property name="text" >
-      <string>Field calculator expression:</string>
+      <string>Field calculator expression</string>
      </property>
+     <property name="buddy" >
+      <cstring>mExpressionTextEdit</cstring>
+     </property>
     </widget>
    </item>
    <item row="6" column="0" colspan="3" >
     <widget class="QTextEdit" name="mExpressionTextEdit" />
    </item>
-   <item row="7" column="0" colspan="2" >
+   <item row="7" column="0" colspan="3" >
     <widget class="QDialogButtonBox" name="mButtonBox" >
      <property name="orientation" >
       <enum>Qt::Horizontal</enum>
      </property>
      <property name="standardButtons" >
-      <set>QDialogButtonBox::Cancel|QDialogButtonBox::Ok</set>
+      <set>QDialogButtonBox::Cancel|QDialogButtonBox::NoButton|QDialogButtonBox::Ok</set>
      </property>
     </widget>
    </item>