From: Krzysztof Kosiński <tweenk.pl@gmail.com>
Date: Tue, 12 Oct 2010 16:23:52 +0000 (+0200)
Subject: Remove the misfeature of retracting handles when the cusp node button
X-Git-Url: https://git.tokkee.org/?a=commitdiff_plain;h=debac3ae63a8499090d97c0d3a6d99b7de4cdfb1;p=inkscape.git

Remove the misfeature of retracting handles when the cusp node button
is clicked and there are cusp nodes selected. It's really annoying when
you have both smooth and cusp nodes in a selection. Use segment commands
and Ctrl+click to retract handles instead.
---

diff --git a/src/ui/tool/node.cpp b/src/ui/tool/node.cpp
index 7efb6a5dc..399fa5292 100644
--- a/src/ui/tool/node.cpp
+++ b/src/ui/tool/node.cpp
@@ -569,10 +569,13 @@ void Node::setType(NodeType type, bool update_handles)
         switch (type) {
         case NODE_CUSP:
             // if the existing type is also NODE_CUSP, retract handles
-            if (_type == NODE_CUSP) {
-                _front.retract();
-                _back.retract();
-            }
+            // NOTE: This misfeature is very annoying when you have both cusp and smooth
+            //       nodes in a selection, so I have removed it. Use segment commands
+            //       or Ctrl+click to retract handles.
+            //if (_type == NODE_CUSP) {
+            //    _front.retract();
+            //    _back.retract();
+            //}
             break;
         case NODE_AUTO:
             // auto handles make no sense for endnodes