Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#1937 closed enhancement (fixed)

[PATCH] Finish last step of modeless editing: Draw command

Reported by: anonymous Owned by: framm
Priority: minor Milestone:
Component: Core Version:
Keywords: Cc:

Description

Currently only one step for modeless editing is missing: The draw command.

A possible suggestion would be

  • call draw command
  • draw nodes and ways until the last node is pressed twice which ends drawing.

This Potlatch like style would allow to draw nodes and ways easily.

Attachments (4)

modelessdrawing.patch (8.2 KB ) - added by xeen 15 years ago.
take that potlatch.patch (3.4 KB ) - added by xeen 15 years ago.
take that potlatch.2.patch (3.7 KB ) - added by xeen 15 years ago.
I'm new to Eclipse
look ma, no modes!.patch (6.2 KB ) - added by xeen 15 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 by anonymous, 15 years ago

Summary: Finish lst step of modeless editing: Draw commandFinish last step of modeless editing: Draw command

comment:2 by stoecker, 15 years ago

Type: defectenhancement

by xeen, 15 years ago

Attachment: modelessdrawing.patch added

comment:4 by xeen, 15 years ago

Summary: Finish last step of modeless editing: Draw command[PATCH] Finish last step of modeless editing: Draw command
  • Adds convenience functions firstNode(), lastNode() and isFirstLastNode() to the way primitive which makes code more readable.
  • Added check to prevent ways like <---> (for more detail, see comment in patch)
  • Will mark a way "finished" if last node is clicked again or a double (triple, …) click is performed.
  • To make the last change actually useful, it will clear the selection if it can't "continue" from it and start anew instead

I did a quick test with all the cases I could make up and it worked fine.

comment:5 by anonymous, 15 years ago

We also need a command for this which returns to select mode after finishing to get really modeless (i.e. like DELETE key compared to delete mode).

comment:6 by xeen, 15 years ago

Yeah going to select mode when a way is finished and going to draw mode if clicking no-where in select mode should pretty much make it modeless. The only case left is where a node is selected and the user wants to draw a way from it. Potlatch uses "Shift+Click" to do this, but my take is that switching draw modes (i.e. pressing "a") is on par here.

Still, one step at a time, this is no trivial change.

comment:7 by anonymous, 15 years ago

Applied in r1400.

comment:8 by anonymous, 15 years ago

Summary: [PATCH] Finish last step of modeless editing: Draw commandFinish last step of modeless editing: Draw command

by xeen, 15 years ago

Attachment: take that potlatch.patch added

by xeen, 15 years ago

Attachment: take that potlatch.2.patch added

I'm new to Eclipse

comment:9 by xeen, 15 years ago

This makes draw/select behave like Potlatch and thus really modeless. It also fixes the "doesn't work for nodes" bug if it's enabled. I'd prefer to keep mode drawing at least as hidden variable, so I'll see if I can fix that, too.

by xeen, 15 years ago

Attachment: look ma, no modes!.patch added

comment:10 by xeen, 15 years ago

Summary: Finish last step of modeless editing: Draw command[PATCH] Finish last step of modeless editing: Draw command

This patch makes mode drawing as well as modeless drawing work fine. Adding single nodes by double clicking now works for both. If you want to continue drawing from a selected mode, you need to select draw mode then click (Potlatch: Shift+Click).

Normally I'd prefer using the same modifier keys, but shift is already taken for add to selection which is a nice and important feature, too. Pressing A, then Click is equal to holding Shift and then Click so that's no disadvantage.

Modeless defaults to off and should be for a while in case errors or bad behaviour are found. We could announce it in the MOTD so get feedback from interested users, though.

comment:11 by anonymous, 15 years ago

Resolution: fixed
Status: newclosed

I added a preferences option for it :-)

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.