#383 closed enhancement (fixed)
[PATCH] auto-weld
Reported by: | Ben | Owned by: | framm |
---|---|---|---|
Priority: | minor | Milestone: | |
Component: | Core | Version: | tested |
Keywords: | Cc: |
Description
Please can there be an option to tern auto weld off? Some may have a use for it, but I havn't met them! Thanks.
(by auto-weld I mean when you 'insert a node' into 2 close by ways, they both get effected.
It may be better to only have it happen when holding shift or something?
Attachments (2)
Change History (10)
comment:1 by , 17 years ago
comment:2 by , 17 years ago
I had read that previously. The problem is that 99% of the time you don't want to weld, and a tiny percentage of the time you do. Different editors may disagree, but the next point applies to all:
If auto-weld is the default and shift is used to stop it then you would commonly 'click, ctrl-z, shift click, as you would only hold shift after reverting as you see that its welded. Thats 5 keys for 1 common job, 1 click for 1 rare job.
comment:3 by , 16 years ago
Component: | unspecified → Core |
---|
comment:4 by , 16 years ago
Owner: | changed from | to
---|
comment:5 by , 16 years ago
We don't have any modifier keys left (CTRL don't connect, SHIFT new way). We could
- Add it as a preference
- Drop shift (not sure the new "doubleclick to finish" is an equal replacement)
IF we drop shift, should it be default-off? Apart from fixing overlapping streets (typical novice mistake) I can't think of any use for auto-weld. It doesn't help novice either because it's very unlikely that they create overlapping streets and then connect a 3rd way/street there.
by , 16 years ago
Attachment: | Update DrawAction.patch added |
---|
comment:7 by , 16 years ago
Summary: | auto-weld → [PATCH] auto-weld |
---|
- SHIFT is now "turn off node reuse" instead of "turn off connection"
- Double clicking/clicking last node again now finishes drawing even if potlatch style drawing is disabled
- ALT key + insert new node in selected way does not produce "bad" selection anymore
- Reworked status line help texts
- a bit of refactoring to make mouseClicked() easier to read
by , 16 years ago
Attachment: | Update DrawAction.2.patch added |
---|
Found and fixed null pointer exception. Otherwise the same.
See: http://lists.openstreetmap.org/pipermail/josm-dev/2007-October/000033.html