Modify

Opened 4 years ago

Closed 4 years ago

Last modified 4 years ago

#19254 closed enhancement (fixed)

History browser: double click relation member table to open history of member

Reported by: simon04 Owned by: simon04
Priority: normal Milestone: 20.05
Component: Core Version:
Keywords: history browser Cc:

Description


Attachments (0)

Change History (13)

comment:1 by simon04, 4 years ago

Resolution: fixed
Status: assignedclosed

In 16459/josm:

fix #19254 - History browser: double click relation member table to open history of member

comment:2 by GerdP, 4 years ago

Resolution: fixed
Status: closedreopened

Good idea, but doesn't work for me. I see in the log that the history of a way is downloaded but there is no corresponding dialog. Instead the view changes from member list to tags for the relation history dialog where I double clicked the member.
Try relation 148838 with double click on member 445220771 (top in v614).
It seems to work when I click on a node in the history for a way, but that also changes the view back to the initial tags view.

comment:3 by GerdP, 4 years ago

See also org.openstreetmap.josm.gui.dialogs.changeset.ChangesetContentPanel.ShowHistoryAction

comment:4 by GerdP, 4 years ago

Found a few more problems:

  • double click on first row is ignored (<= 0 should be < 0)
  • javadoc is wrong

Working on a patch ...

comment:5 by GerdP, 4 years ago

In 16460/josm:

see #19254

  • double click on first row did not work
  • allow to use all implementations of PrimitiveId in HistoryDataSet.getHistory()
  • simplify code, correct javadoc

comment:6 by GerdP, 4 years ago

Resolution: fixed
Status: reopenedclosed

comment:7 by GerdP, 4 years ago

We have now three implementations of a ShowHistoryAction, all more or less doing the same. I am sure this can be simplified.
See #19257

Last edited 4 years ago by GerdP (previous) (diff)

comment:8 by GerdP, 4 years ago

Resolution: fixed
Status: closedreopened

The original dialog still switches the view to the tags panel. No idea why.

in reply to:  7 comment:9 by simon04, 4 years ago

Thanks for addressing the issues you found!

Replying to GerdP:

We have now three implementations of a ShowHistoryAction.

I haven't added one, but just moved it. *washing my hands of responsibility* :-)
A harmonization is very welcome.

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

Replying to GerdP:

The original dialog still switches the view to the tags panel. No idea why.

I tried to find the code point where this happens but failed. It is not related to this ticket, same problem occurs when you select a different object and press Ctrl+H.

comment:11 by GerdP, 4 years ago

Resolution: fixed
Status: reopenedclosed

in reply to:  10 ; comment:12 by skyper, 4 years ago

Replying to GerdP:

Replying to GerdP:

The original dialog still switches the view to the tags panel. No idea why.

I tried to find the code point where this happens but failed. It is not related to this ticket, same problem occurs when you select a different object and press Ctrl+H.

Still present and annoying as it can complete break the workflow. ATM, only a feature under certain circumstances but a bug in other situations.

in reply to:  12 comment:13 by GerdP, 4 years ago

Replying to skyper:

Replying to GerdP:

Replying to GerdP:

The original dialog still switches the view to the tags panel. No idea why.

I tried to find the code point where this happens but failed. It is not related to this ticket, same problem occurs when you select a different object and press Ctrl+H.

Still present and annoying as it can complete break the workflow. ATM, only a feature under certain circumstances but a bug in other situations.

Fixed in #19367

Modify Ticket

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