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.

Author: nyalldawson

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();