Modify

Opened 8 years ago

Last modified 15 months ago

#15489 new defect

Action "Duplicate" places new way too far away

Reported by: GerdP Owned by: team
Priority: normal Milestone:
Component: Core Version:
Keywords: template_report Cc:

Description

What steps will reproduce the problem?

  1. Select a way with many nodes, e.g. way 58202859
  2. Press Ctrl+D to duplicate it
  3. The duplicated way is add so far away that I had to zoom out to find it.

What is the expected result?

in the past it was added much closer to the original

What happens instead?

Please provide any additional information below. Attach a screenshot if possible.

URL:http://josm.openstreetmap.de/svn/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2017-10-03 21:43:00 +0200 (Tue, 03 Oct 2017)
Build-Date:2017-10-03 19:50:21
Revision:12921
Relative:URL: ^/trunk

Identification: JOSM/1.5 (12921 en) Windows 10 64-Bit
OS Build number: Windows 10 Home 1703 (15063)
Memory Usage: 1671 MB / 4444 MB (1315 MB allocated, but free)
Java version: 1.8.0_111-b14, Oracle Corporation, Java HotSpot(TM) 64-Bit Server VM
Screen: \Display0 1920x1080
Maximum Screen Size: 1920x1080
Dataset consistency test: No problems found

Plugins:
+ FastDraw (33583)
+ apache-commons (33668)
+ buildings_tools (33004)
+ download_along (33710)
+ ejml (32680)
+ geotools (33380)
+ jts (32699)
+ measurement (33088)
+ o5m (33566)
+ opendata (33617)
+ pbf (33568)
+ poly (33570)
+ reverter (33572)
+ utilsplugin2 (33704)

Map paint styles:
- https://josm.openstreetmap.de/josmfile?page=Styles/Lane_and_Road_Attributes&zip=1

Last errors/warnings:
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet
- W: java.io.IOException: Attribution is not loaded yet

Attachments (0)

Change History (6)

comment:1 by Klumbumbus, 8 years ago

It usually placed the duplicate at the mouse position. (I didn't check what happens if the mouse is out of the mapview.)

comment:2 by GerdP, 8 years ago

Yes, now it seems to place the object so that the middle of the bbox is at the cursor. With a curvy way this is not a good idea.

comment:3 by GerdP, 8 years ago

I am not sure if I am mislead. I seem to remember that a duplicated way was selected and when I moved the cursor the geometry of the way was adapted. Or was that a different function? It was useful when you want to split a wood polygon along a major road.

comment:4 by Klumbumbus, 8 years ago

Do you mean Help/Action/Parallel?

in reply to:  4 comment:5 by GerdP, 8 years ago

Replying to Klumbumbus:

Do you mean Help/Action/Parallel?

Yes! Thanks for the hint. So the duplicate action probably always worked like now. Still, it doesn't work well for my example.
I think there should be some code that makes sure that the pasted object is at least visible, means, at least one segment should cross the current visible area, but maybe other users are used to the current behaviour.

comment:6 by GerdP, 15 months ago

I didn't mention before that one has to zoom in so that only a small part of the selected way is visible, e.g. zoom on node 72297281.
I think a similar problem occurs when you intend to paste the tags of another object to a new way. One way to do that is to select the way and press Ctrl+C and then select the new way and press Ctrl+Shift+V. Sometimes I accidentially press only Ctrl+V and thus paste the way instead of its tags. The user feedback would be that the selected way is unselected and the (invisible) pasted way is selected,
but the wanted tags will appear.

I think it would be good when JOSM shows a notification like "data was pasted outside of the visible area" or just zoom out so that the pasted object is fully visible, but I have no idea how to detect the problematic case.

Modify Ticket

Change Properties
Set your email in Preferences
Action
as new The owner will remain team.
as The resolution will be set. Next status will be 'closed'.
to The owner will be changed from team to the specified user.
Next status will be 'needinfo'. The owner will be changed from team to GerdP.
as duplicate The resolution will be set to duplicate. Next status will be 'closed'. The specified ticket will be cross-referenced with this ticket.
The owner will be changed from team to anonymous. Next status will be 'assigned'.

Add Comment


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