Modify

Opened 17 years ago

Closed 16 years ago

Last modified 15 years ago

#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)

Update DrawAction.patch (19.9 KB ) - added by xeen 16 years ago.
Update DrawAction.2.patch (19.4 KB ) - added by xeen 16 years ago.
Found and fixed null pointer exception. Otherwise the same.

Download all attachments as: .zip

Change History (10)

comment:2 by Ben, 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 stoecker, 16 years ago

Component: unspecifiedCore

comment:4 by stoecker, 16 years ago

Owner: changed from imi to framm

comment:5 by xeen, 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.

comment:6 by xeen, 16 years ago

Ticket #474 has been marked as a duplicate of this ticket.

by xeen, 16 years ago

Attachment: Update DrawAction.patch added

comment:7 by xeen, 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 xeen, 16 years ago

Attachment: Update DrawAction.2.patch added

Found and fixed null pointer exception. Otherwise the same.

comment:8 by stoecker, 16 years ago

Resolution: fixed
Status: newclosed

Applied in r1438.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain framm.
as The resolution will be set.
The resolution will be deleted. Next status will be 'reopened'.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.