geometry less features: update add feature tool in digitizing toolbar…

… and icon in attribute table (fixes #8262)
qgis · Feb 13, 2014 · dae8ea6 · dae8ea6
1 parent 588d5f4
commit dae8ea6
Show file tree

Hide file tree

Showing 7 changed files with 32 additions and 1 deletion.
diff --git a/images/images.qrc b/images/images.qrc
@@ -147,6 +147,7 @@
         <file>themes/default/mActionCaptureLine.png</file>
         <file>themes/default/mActionCapturePoint.png</file>
         <file>themes/default/mActionCapturePolygon.png</file>
+        <file>themes/default/mActionNewTableRow.png</file>
         <file>themes/default/mActionChangeLabelProperties.png</file>
         <file>themes/default/mActionCheckQgisVersion.png</file>
         <file>themes/default/mActionCollapseTree.png</file>

diff --git a/images/themes/default/mActionNewTableRow.png b/images/themes/default/mActionNewTableRow.png
diff --git a/src/app/qgisapp.cpp b/src/app/qgisapp.cpp
@@ -8701,6 +8701,10 @@ void QgisApp::activateDeactivateLayerRelatedActions( QgsMapLayer* layer )
         mActionDeleteRing->setEnabled( isEditable && canChangeGeometry );
         mActionOffsetCurve->setEnabled( false );
       }
+      else if ( vlayer->geometryType() == QGis::NoGeometry )
+      {
+        mActionAddFeature->setIcon( QgsApplication::getThemeIcon( "/mActionNewTableRow.png" ) );
+      }
 
       mActionOpenFieldCalc->setEnabled( isEditable && ( canChangeAttributes || canAddAttributes ) );
 

diff --git a/src/app/qgsattributetabledialog.cpp b/src/app/qgsattributetabledialog.cpp
@@ -154,6 +154,7 @@ QgsAttributeTableDialog::QgsAttributeTableDialog( QgsVectorLayer *theLayer, QWid
   mTableViewButton->setIcon( QgsApplication::getThemeIcon( "/mActionOpenTable.png" ) );
   mAttributeViewButton->setIcon( QgsApplication::getThemeIcon( "/mActionPropertyItem.png" ) );
   mExpressionSelectButton->setIcon( QgsApplication::getThemeIcon( "/mIconExpressionSelect.svg" ) );
+  mAddFeature->setIcon( QgsApplication::getThemeIcon( "/mActionNewTableRow.png" ) );
 
   // toggle editing
   bool canChangeAttributes = mLayer->dataProvider()->capabilities() & QgsVectorDataProvider::ChangeAttributeValues;

diff --git a/src/app/qgsmaptooladdfeature.cpp b/src/app/qgsmaptooladdfeature.cpp
@@ -45,6 +45,24 @@ bool QgsMapToolAddFeature::addFeature( QgsVectorLayer *vlayer, QgsFeature *f )
   return action.addFeature();
 }
 
+void QgsMapToolAddFeature::activate()
+{
+  if ( !mCanvas || mCanvas->isDrawing() )
+  {
+    return;
+  }
+
+  QgsVectorLayer *vlayer = qobject_cast<QgsVectorLayer *>( mCanvas->currentLayer() );
+  if ( vlayer && vlayer->geometryType() == QGis::NoGeometry )
+  {
+    QgsFeature f;
+    addFeature( vlayer, &f );
+    return;
+  }
+
+  QgsMapTool::activate();
+}
+
 void QgsMapToolAddFeature::canvasReleaseEvent( QMouseEvent * e )
 {
   QgsDebugMsg( "entered." );

diff --git a/src/app/qgsmaptooladdfeature.h b/src/app/qgsmaptooladdfeature.h
@@ -26,4 +26,5 @@ class APP_EXPORT QgsMapToolAddFeature : public QgsMapToolCapture
     void canvasReleaseEvent( QMouseEvent * e );
 
     bool addFeature( QgsVectorLayer *vlayer, QgsFeature *f );
+    void activate();
 };
diff --git a/src/ui/qgsattributetabledialog.ui b/src/ui/qgsattributetabledialog.ui
@@ -105,7 +105,13 @@
         <string>Add feature</string>
        </property>
        <property name="text">
-        <string>+</string>
+        <string/>
+       </property>
+       <property name="iconSize">
+        <size>
+         <width>18</width>
+         <height>18</height>
+        </size>
        </property>
       </widget>
      </item>