Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#1937 closed enhancement (fixed)

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

Reported by: anonymous Owned by: framm
Priority: minor 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 4 years ago.
take that potlatch.patch (3.4 KB) - added by xeen 4 years ago.
take that potlatch.2.patch (3.7 KB) - added by xeen 4 years ago.
I'm new to Eclipse
look ma, no modes!.patch (6.2 KB) - added by xeen 4 years ago.

Download all attachments as: .zip

Change History (14)

comment:1 Changed 4 years ago by anonymous

  • Summary changed from Finish lst step of modeless editing: Draw command to Finish last step of modeless editing: Draw command

comment:2 Changed 4 years ago by stoecker

  • Type changed from defect to enhancement

Changed 4 years ago by xeen

comment:4 Changed 4 years ago by xeen

  • Cc WChrMuhWAdU removed
  • Keywords JLtVjPCtYFzw removed
  • Summary changed from Finish last step of modeless editing: Draw command to [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 Changed 4 years ago by anonymous

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 Changed 4 years ago by xeen

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 Changed 4 years ago by anonymous

Applied in r1400.

comment:8 Changed 4 years ago by anonymous

  • Summary changed from [PATCH] Finish last step of modeless editing: Draw command to Finish last step of modeless editing: Draw command

Changed 4 years ago by xeen

Changed 4 years ago by xeen

I'm new to Eclipse

comment:9 Changed 4 years ago by xeen

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.

Changed 4 years ago by xeen

comment:10 Changed 4 years ago by xeen

  • Summary changed from Finish last step of modeless editing: Draw command to [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 Changed 4 years ago by anonymous

  • Resolution set to fixed
  • Status changed from new to closed

I added a preferences option for it :-)

Add Comment

Modify Ticket

Change Properties
<Author field>
Action
as closed .
as The resolution will be set. Next status will be 'closed'.
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.