When deleting nodes from the end of a linestring, select the

final node following a delete rather than reverting to the first node. Improves workflow when deleting sequential nodes from end of lines. (cherry-picked from bcb7e44)
qgis · Nov 18, 2015 · f679daa · f679daa
1 parent 1449078
commit f679daa
Showing 1 changed file with 8 additions and 1 deletion.
diff --git a/src/app/nodetool/qgsmaptoolnodetool.cpp b/src/app/nodetool/qgsmaptoolnodetool.cpp
@@ -546,7 +546,14 @@ void QgsMapToolNodeTool::keyPressEvent( QKeyEvent* e )
     }
     else
     {
-      safeSelectVertex( firstSelectedIndex );
+      int nextVertexToSelect = firstSelectedIndex;
+      if ( mSelectedFeature->geometry()->type() == QGis::Line )
+      {
+        // for lines we don't wrap around vertex selection when deleting nodes from end of line
+        nextVertexToSelect = qMin( nextVertexToSelect, mSelectedFeature->geometry()->geometry()->nCoordinates() - 1 );
+      }
+
+      safeSelectVertex( nextVertexToSelect );
     }
     mCanvas->refresh();