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 7efb6a5dc87f6935b0ccccd2b76c651d45a95614..399fa529238bfd149dd3b9100493377972bbfb48 100644 (file)
--- 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