Opened 15 years ago
Closed 13 years ago
#4549 closed defect (fixed)
"Join Node to Way" should be called "Join Way to Node"
Reported by: | Owned by: | team | |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | Core | Version: | latest |
Keywords: | Cc: |
Description (last modified by )
When you select a way y and a node x like this:
y | | x---- | y
"Join Node to Way" will actually move the way y to include node x. Hence it should be called "Join Way to Node" and not "Join Node to Way" since it's moving the way to the node, and not the other way around.
Attachments (1)
Change History (9)
follow-ups: 2 3 comment:1 by , 15 years ago
comment:2 by , 15 years ago
Replying to mjulius:
I don't agree. A way is a list of nodes and the new node is joined to that list. Maybe it would be more accurate to call this "Include Node in Way", but I don't think this is worth the trouble.
You seem to be confusing internal details with UI usability and expectancy. Internal details are of no importance whatsoever to a mapper doing some editing.
I expect 'Join node to way' to move the node to be on top of the way and add it to the way.
If I want to do the reverse, I grab a virtual node (another misnomer) from the way and ctrl-drop it onto the node. Don't need 'Join node to way' to perform that.
follow-up: 5 comment:3 by , 15 years ago
Replying to mjulius:
I don't agree. A way is a list of nodes and the new node is joined to that list. Maybe it would be more accurate to call this "Include Node in Way", but I don't think this is worth the trouble.
I can see your point but as Ldp points out a user is more likely to think about this visually. I.e. "Join Node to Way" would imply that Node is to be moved to where Way is now. But "Include Node in Way" would be less ambiguous and seems like a better solution.
Actually what would really help the usability is if the icon had a little arrow indicating which item is the one that's going to be moved. Unfortunately I don't have the graphics skills to submit one.
by , 13 years ago
Attachment: | joinnodeway.png added |
---|
comment:5 by , 13 years ago
Description: | modified (diff) |
---|
Replying to avar:
Actually what would really help the usability is if the icon had a little arrow indicating which item is the one that's going to be moved.
I tried to add an arrow to the icon: attachment:joinnodeway.png
"Include Node in Way" is unambiguous, however, it makes the shortcut unintuitive …
comment:8 by , 13 years ago
Resolution: | → fixed |
---|---|
Status: | new → closed |
I don't agree. A way is a list of nodes and the new node is joined to that list. Maybe it would be more accurate to call this "Include Node in Way", but I don't think this is worth the trouble.