Modify

Opened 7 years ago

Closed 7 years ago

Last modified 7 years ago

#14708 closed defect (fixed)

no warning about potentially erroneous movement of existing nodes

Reported by: Don-vip Owned by: team
Priority: normal Milestone: 17.04
Component: Core Version:
Keywords: Cc: planemad

Description

from 11405 comments:

The warning works on newly created geometries but does not show up when moving the node for an existing object.

https://cloud.githubusercontent.com/assets/126868/25421121/f5ea3e0a-2a78-11e7-96ba-16587deacb5c.gif

Attachments (1)

patch-14708.patch (4.4 KB ) - added by michael2402 7 years ago.

Download all attachments as: .zip

Change History (6)

comment:1 by Don-vip, 7 years ago

Resolution: fixed
Status: newclosed

In 12032/josm:

fix #14708 - no warning about potentially erroneous movement of existing nodes

by michael2402, 7 years ago

Attachment: patch-14708.patch added

comment:2 by michael2402, 7 years ago

You were faster than me.

I added a test for the way segment to highlight the exact segment that causes the problem. I'll merge it after the release.

comment:3 by Don-vip, 7 years ago

You can merge it, I'll do the release after the i18n update.

comment:4 by michael2402, 7 years ago

In 12037/josm:

See #14708: For long segment test, notify the user about the exact segment. Do not check segments that have not been touched.

comment:5 by michael2402, 7 years ago

Zoom to problem now works by zooming to the right way segment ;-).

Only disadvantage: You get a warning for each way segment. They are grouped by segment length, not by way, so when working with a lot of ways that contain long segment (borders, ...), you might get a lot of messages. But people working there hopefully know what they are doing.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain team.
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.