On 21-4-2012 22:58, Diederik van Lierop wrote:
On 04/17/2012 10:33 PM, Johan Engelen wrote:
On 17-4-2012 21:57, Diederik van Lierop wrote:
And that one should snap, always, regardless of type. Don't you find it very awkward to see some other node snap than the one you are dragging? Why wouldn't you just have dragged that node instead?
The node tool is indeed different in that respect from the selector tool. So we could make the node tool behave differently. But I hate all these small inconsistencies!
No this is not confusing at all, since the selector tool cannot see nodes, nor can you "select them and drag them". It is quite different really.
Yes, every tool is different, I agree. But let's please try to make all tools use the settings, preferences, and shortcut-keys in the exact same way! Different logic for different tools is bad practice.
It is not different logic or inconsistency. It is as consistent as showing the nodes only in the node tool, showing bbox only in selection tool, showing circle nodes in circle tool, etc.
I don't agree with you on this point. Showing different elements (e.g. nodes, handles, bounding boxes, etc) for different tools indeed makes sense. However, this does not apply for having different behavior per tool. For one, showing different things is very explicit and is hard to miss. Imagine however that someone would like to disable snapping in the node tool. In each and every tool, this can be achieved using either of three methods:
- by disabling the global snap toggle, or
- by disabling the "snap nodes, paths, and handles" group, or
- by disabling both the "snap cusp nodes" and "snap smooth nodes" group.
Now if we would hard code snapping of nodes in the node tool, then the latter method would not work. Sure, our users have two other options left, but there will be people running into this, trying to disable snapping using the third methods and not succeeding.
And in the end one might even file a bug report for this.
Really? Did that ever happen with the past versions? We already have bug reports about the current UI...
Apparently, I cannot stop you guys from adding this extremely fine-detailed control of snapping. Please do not add it anywhere in the toolbar UI, but hide it somewhere deep instead, and set it to the correct (nota bene) values per default. Choice of snap targets is interesting and should be in UI, choice of snap sources like nodes isn't.
(other examples of toggles ready for removal: grid and guides snapping toggles. shift+3, shift+| work fine for disabling the showing/snapping of grids/guides. yes you won't be able to see them but... come on.... Perpendicular and tangential snap toggles would be a much better of that space)
Cheers, Johan