From debac3ae63a8499090d97c0d3a6d99b7de4cdfb1 Mon Sep 17 00:00:00 2001 From: =?utf8?q?Krzysztof=20Kosi=C5=84ski?= Date: Tue, 12 Oct 2010 18:23:52 +0200 Subject: [PATCH] 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. --- src/ui/tool/node.cpp | 11 +++++++---- 1 file changed, 7 insertions(+), 4 deletions(-) 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 -- 2.30.2