Snap while rotating: fix removal of points too close to the rotation center

(bzr r10093)

1 files changed, 12 insertions, 12 deletions

diff --git a/src/snap.cpp b/src/snap.cpp
index f13b02b46..fb8f70c49 100644
--- a/src/snap.cpp
+++ b/src/snap.cpp
@@ -760,16 +760,6 @@ Inkscape::SnappedPoint SnapManager::_snapTransformed(
                 dedicated_constraint = Inkscape::Snapper::SnapConstraint(origin, b);
             } else if (transformation_type == ROTATE) {
                 Geom::Coord r = Geom::L2(b); // the radius of the circular constraint
-                if (r < 1e-9) { // points too close to the rotation center will not move. Don't try to snap these
-                    // as they will always yield a perfect snap result if they're already snapped beforehand (e.g.
-                    // when the transformation center has been snapped to a grid intersection in the selector tool)
-                    continue; // skip this SnapCandidate and continue with the next one
-                    // PS1: Apparently we don't have to do this for skewing, but why?
-                    // PS2: We cannot easily filter these points upstream, e.g. in the grab() method (seltrans.cpp)
-                    // because the rotation center will change when pressing shift, and grab() won't be recalled.
-                    // Filtering could be done in handleRequest() (again in seltrans.cpp), by iterating through
-                    // the snap candidates. But hey, we're iterating here anyway.
-                }
                 dedicated_constraint = Inkscape::Snapper::SnapConstraint(origin, b, r);
             } else if (transformation_type == STRETCH) { // when non-uniform stretching {
                 dedicated_constraint = Inkscape::Snapper::SnapConstraint((*i).getPoint(), component_vectors[dim]);
@@ -895,8 +885,18 @@ Inkscape::SnappedPoint SnapManager::_snapTransformed(
                 // a is vector to snapped point; b is vector to original point; now lets calculate angle between a and b
                 result[0] = atan2(Geom::dot(Geom::rot90(b), a), Geom::dot(b, a));
                 result[1] = result[1]; // how else should we store an angle in a point ;-)
-                // Store the metric for this transformation as a virtual distance (we're storing an angle)
-                snapped_point.setSnapDistance(std::abs(result[0] - transformation[0]));
+                if (Geom::L2(b) < 1e-9) { // points too close to the rotation center will not move. Don't try to snap these
+                    // as they will always yield a perfect snap result if they're already snapped beforehand (e.g.
+                    // when the transformation center has been snapped to a grid intersection in the selector tool)
+                    snapped_point.setSnapDistance(NR_HUGE);
+                    // PS1: Apparently we don't have to do this for skewing, but why?
+                    // PS2: We cannot easily filter these points upstream, e.g. in the grab() method (seltrans.cpp)
+                    // because the rotation center will change when pressing shift, and grab() won't be recalled.
+                    // Filtering could be done in handleRequest() (again in seltrans.cpp), by iterating through
+                    // the snap candidates. But hey, we're iterating here anyway.
+                } else {
+                    snapped_point.setSnapDistance(std::abs(result[0] - transformation[0]));
+                }
                 snapped_point.setSecondSnapDistance(NR_HUGE);
                 break;
             default: