Modify ↓

Opened 3 years ago

Closed 3 years ago

Last modified 11 months ago

#20899 closed defect (fixed)

[patch] Improve Way Accuracy Action: Warn when moving elements by a large distance

Reported by: ljdelight Owned by: GerdP
Priority: normal Milestone: 21.10
Component: Core Version:
Keywords: improve way accuracy move node Cc:

Description

A warning dialog is shown when dragging nodes a large distance, but this dialog does not appear with the improve way accuracy W-shortcut.
This patch updates the W-shortcut logic to check for nodes that move a large distance and show a warning dialog.

Attachments (1)

w-confirmMovement.patch​ (4.4 KB ) - added by ljdelight 3 years ago.

Download all attachments as: .zip

Change History (12)

by ljdelight, 3 years ago

Attachment: w-confirmMovement.patch​ added

comment:1 by ljdelight, 3 years ago

Hi, what are your thoughts on the patch? Thank you

comment:2 by ljdelight, 3 years ago

Hi @simon04 and @GerdP, let me know if you have any suggestions for the patch. Thanks

comment:3 by ljdelight, 3 years ago

Milestone: → 21.09

comment:4 by GerdP, 3 years ago

Milestone: 21.09 → 21.10
Owner: changed from team to GerdP
Status: new → assigned

Sorry for the late response. Patch looks good to me, will commit it after the release of 21.09

comment:5 by skyper, 3 years ago

It would be nice if the ImproveWay plugin give the same warning.

comment:6 by GerdP, 3 years ago

Can be done once this patch was committed.

comment:7 by Don-vip, 3 years ago

Milestone: 21.10 → 21.11

Milestone renamed

comment:8 by GerdP, 3 years ago

Resolution: → fixed
Status: assigned → closed

In 18274/josm:

fix #20899: Improve Way Accuracy Action: Warn when moving elements by a large distance
(patch w-confirmMovement.patch by ljdelight)
This patch updates the W-shortcut logic to check for nodes that move a large distance and show a warning dialog.

comment:9 by GerdP, 3 years ago

Milestone: 21.11 → 21.10

in reply to:  6 comment:10 by skyper, 3 years ago

Replying to GerdP:

Can be done once this patch was committed.

#21506

comment:11 by taylor.smock, 11 months ago

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

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain GerdP.
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 .
Attachments ↑ Description ↑
 
Note: See TracTickets for help on using tickets.