On 2/11/10 00:29, Krzysztof Kosiński wrote:
W dniu 1 listopada 2010 20:27 użytkownik ~suv <suv-sf@...58...> napisał:
If handles will only be retracted when all selected nodes are cusp, does it still allow to convert a selection of mostly smooth nodes to cusp nodes?
If yes, then after using the command once, all nodes are cusp now (though some still have colinear handles), and another 'Shift+C' would retract all handles -> same as previous behavior.
Yes, this will be the new behavior. Previously, when you selected a cusp and a smooth node, clicking on the cusp node button would convert the smooth node to cusp and simultaneously retract the handles of the cusp node. (Probably the old node tool behaved this way, but the new one didn't)
My logic module must have had low battery yesterday ;)
So, the difference I failed to grasp between the behavior as seen in 0.47 / 0.48 and your proposed compromise is about the 'intermediary' state of cusp nodes with extracted handles (joining two curves or a curve and a line):
A -- previous behavior with a selection of mixed nodes types:
Shift+C Shift+C initial node type | | | | | smooth (h visible) --> cusp (h unchanged) --> cusp (h retracted) cusp (h extracted) --> cusp (h retracted) --> cusp (h no-op) cusp (h retracted) --> cusp (h no-op) --> cusp (h no-op)
B -- proposed behavior with a selection of mixed nodes types:
Shift+C Shift+C initial node type | | | | | smooth (h visible) --> cusp (h unchanged) --> cusp (h retracted) cusp (h extracted) --> cusp (h no-op) --> cusp (h retracted) cusp (h retracted) --> cusp (h no-op) --> cusp (h no-op)
I think this would be fine (tests with the actual tool outstanding), but certainly it would be appreciated to have the feature to retract the handles of multiple selected nodes restored.
Thanks for looking into this, ~suv
p.s.
Possibly a slightly related issue: Bug #655222 “Tangent node doesn't remain smooth after removing control handles”: https://bugs.launchpad.net/inkscape/+bug/610817 also reported in https://bugs.launchpad.net/inkscape/+bug/610817
... and in case you have some spare time ;) list of reports tagged with 'node-editing' sorted by 'newest first': https://bugs.launchpad.net/inkscape/+bugs?field.tag=node-editing&orderby=-datecreated&field.omit_dupes.used and a shorter one, tagged with 'node-editing' and 'regression': https://bugs.launchpad.net/inkscape/+bugs?field.tag=node-editing+regression&field.tags_combinator=ALL&orderby=-datecreated&field.omit_dupes.used