From: dvlierop2 <dvlierop2@users.sourceforge.net>
Date: Wed, 21 Mar 2007 20:46:18 +0000 (+0000)
Subject: Fix for bug #1589436 - font baseline now correctly snaps in selector tool
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=21c27920218c0cee0a6231c9fff353dfd5ad74d4;p=inkscape.git

Fix for bug #1589436 - font baseline now correctly snaps in selector tool
---

diff --git a/src/seltrans.cpp b/src/seltrans.cpp
index 150a67c30..5ace790ec 100644
--- a/src/seltrans.cpp
+++ b/src/seltrans.cpp
@@ -255,7 +255,14 @@ void Inkscape::SelTrans::grab(NR::Point const &p, gdouble x, gdouble y, bool sho
 
     _point = p;
 
-    _snap_points = selection->getSnapPointsConvexHull();
+    _snap_points = selection->getSnapPoints();
+    if (_snap_points.size() > 100) {
+        /* Snapping a huge number of nodes will take way too long, so limit the number of snappable nodes
+        An average user would rarely ever try to snap such a large number of nodes anyway, because 
+        (s)he could hardly discern which node would be snapping */
+        _snap_points = selection->getSnapPointsConvexHull();
+        // Unfortunately, by now we will have lost the font-baseline snappoints :-(
+    }
     _box = selection->bounds();
     _bbox_points.clear();
     if (_box) {