Better snapping behavior

leyan · leyan · commit 4768a8fdb6b5 · 2014-05-21T21:12:26.000+08:00
diff --git a/src/app/qgsmaptoolsplitfeatures.cpp b/src/app/qgsmaptoolsplitfeatures.cpp
@@ -58,10 +58,10 @@ void QgsMapToolSplitFeatures::canvasReleaseEvent( QMouseEvent * e )
   {
     QList<QgsSnappingResult> snapResults;
 
-    //If we snap the first point on a line layer, we directly split the feature at this point
+    //If we snap the first point on a vertex of a line layer, we directly split the feature at this point
     if ( vlayer->geometryType() == QGis::Line && points().isEmpty() )
     {
-      if ( mSnapper.snapToBackgroundLayers( e->pos(), snapResults ) == 0 )
+      if ( mSnapper.snapToCurrentLayer( e->pos(), snapResults, QgsSnapper::SnapToVertex ) == 0 )
       {
         if ( snapResults.size() > 0 )
         {
diff --git a/src/app/qgsmaptoolsplitparts.cpp b/src/app/qgsmaptoolsplitparts.cpp
@@ -56,10 +56,10 @@ void QgsMapToolSplitParts::canvasReleaseEvent( QMouseEvent * e )
   {
     QList<QgsSnappingResult> snapResults;
 
-    //If we snap the first point on a line layer, we directly split the feature at this point
+    //If we snap the first point on a vertex of a line layer, we directly split the feature at this point
     if ( vlayer->geometryType() == QGis::Line && points().isEmpty() )
     {
-      if ( mSnapper.snapToBackgroundLayers( e->pos(), snapResults ) == 0 )
+      if ( mSnapper.snapToCurrentLayer( e->pos(), snapResults, QgsSnapper::SnapToVertex ) == 0 )
       {
         if ( snapResults.size() > 0 )
         {

Original file line number	Diff line number	Diff line change
`@@ -58,10 +58,10 @@ void QgsMapToolSplitFeatures::canvasReleaseEvent( QMouseEvent * e )`
`58`	`58`	`{`
`59`	`59`	`QList<QgsSnappingResult> snapResults;`
`60`	`60`
`61`		`- //If we snap the first point on a line layer, we directly split the feature at this point`
	`61`	`+ //If we snap the first point on a vertex of a line layer, we directly split the feature at this point`
`62`	`62`	`if ( vlayer->geometryType() == QGis::Line && points().isEmpty() )`
`63`	`63`	`{`
`64`		`- if ( mSnapper.snapToBackgroundLayers( e->pos(), snapResults ) == 0 )`
	`64`	`+ if ( mSnapper.snapToCurrentLayer( e->pos(), snapResults, QgsSnapper::SnapToVertex ) == 0 )`
`65`	`65`	`{`
`66`	`66`	`if ( snapResults.size() > 0 )`
`67`	`67`	`{`
Original file line number	Diff line number	Diff line change
`@@ -56,10 +56,10 @@ void QgsMapToolSplitParts::canvasReleaseEvent( QMouseEvent * e )`
`56`	`56`	`{`
`57`	`57`	`QList<QgsSnappingResult> snapResults;`
`58`	`58`
`59`		`- //If we snap the first point on a line layer, we directly split the feature at this point`
	`59`	`+ //If we snap the first point on a vertex of a line layer, we directly split the feature at this point`
`60`	`60`	`if ( vlayer->geometryType() == QGis::Line && points().isEmpty() )`
`61`	`61`	`{`
`62`		`- if ( mSnapper.snapToBackgroundLayers( e->pos(), snapResults ) == 0 )`
	`62`	`+ if ( mSnapper.snapToCurrentLayer( e->pos(), snapResults, QgsSnapper::SnapToVertex ) == 0 )`
`63`	`63`	`{`
`64`	`64`	`if ( snapResults.size() > 0 )`
`65`	`65`	`{`