Modify

Opened 7 months ago

Closed 7 months ago

Last modified 7 months ago

#18130 closed defect (fixed)

[Patch] Deleting geotagged image doesn't jump to next image anymore

Reported by: skorbut Owned by: francois2
Priority: normal Milestone: 19.09
Component: Core image mapping Version:
Keywords: template_report regression Cc: francois2

Description (last modified by Don-vip)

What steps will reproduce the problem?

  1. Drag-and-drop a directory of geotagged images into JOSM
  2. Delete one of the images permanently using the disc-with-red-cross-button in the JOSM image viewer

What is the expected result?

The viewer should display the next image in the sequence.

What happens instead?

The viewer doesn't display any image, the control buttons are greyed out, there is a text No image. Other images can still be selected and displayed.

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

This behaviour is quite new, it can only be a few days old. Maybe it's related to the activities in #11905 ?

Relative:URL: ^/trunk
Repository:UUID: 0c6e7542-c601-0410-84e7-c038aed88b3b
Last:Changed Date: 2019-09-12 23:23:47 +0200 (Thu, 12 Sep 2019)
Revision:15348
Build-Date:2019-09-12 21:28:35
URL:https://josm.openstreetmap.de/svn/trunk

... snip ...

Attachments (1)

patch_select_image_after_remove.patch (3.2 KB) - added by francois2 7 months ago.
Select next image after remove

Download all attachments as: .zip

Change History (9)

comment:1 Changed 7 months ago by Don-vip

Cc: francois2 added
Description: modified (diff)
Keywords: regression added
Milestone: 19.09

@francois2 can you please take a look?

comment:2 in reply to:  description Changed 7 months ago by Don-vip

Replying to skorbut:

... snip ...

Please don't do that. Java version and OS information is the most crucial piece of information of status reports.

comment:3 Changed 7 months ago by francois2

I'll take a look tomorrow.

comment:4 Changed 7 months ago by francois2

Owner: changed from team to francois2

Changed 7 months ago by francois2

Select next image after remove

comment:5 Changed 7 months ago by francois2

Summary: Deleting geotagged image doesn't jump to next image anymore[Patch] Deleting geotagged image doesn't jump to next image anymore

comment:6 Changed 7 months ago by Don-vip

Resolution: fixed
Status: newclosed

In 15352/josm:

fix #18130 - Select next image after remove (patch by francois2)

comment:7 Changed 7 months ago by Don-vip

thanks! :)

comment:8 Changed 7 months ago by skorbut

Seems to work fine. Thank you for your fast reaction!

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain francois2.
as The resolution will be set.
The resolution will be deleted.

Add Comment


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

 
Note: See TracTickets for help on using tickets.