Modify

Opened 3 years ago

Closed 3 years ago

Last modified 3 years ago

#21427 closed defect (fixed)

[PATCH] further simplify UploadDialog

Reported by: marcello@… Owned by: team
Priority: normal Milestone: 21.10
Component: Core Version:
Keywords: upload changeset Cc:

Description

This patch simplifies UploadDialog in a further attempt to nail #20690.

  • The dialog was simplified by combining the function of two radiobuttons and one combobox into one combobox.
  • When an open changeset was selected on tab 2, existing tags on the open changeset could overwrite the data the user entered on tab 1. The user might spot this by looking closely at the tag table on tab 2, but then he may not. This non-obvious behaviour was removed.
  • The exception thrown when closing an already closed changeset was fixed.
  • More cosmetic changes to the dialog.

Maybe also fixes #19319, #21387 (added revalidate()).

Attachments (1)

21427.patch (75.0 KB ) - added by marcello@… 3 years ago.

Download all attachments as: .zip

Change History (5)

by marcello@…, 3 years ago

Attachment: 21427.patch added

comment:1 by Don-vip, 3 years ago

Keywords: upload changeset added
Milestone: 21.10

comment:2 by Don-vip, 3 years ago

Resolution: fixed
Status: newclosed

In 18283/josm:

fix #21427 - further simplify UploadDialog (patch by marcello, modified)

  • The dialog was simplified by combining the function of two radiobuttons and one combobox into one combobox.
  • When an open changeset was selected on tab 2, existing tags on the open changeset could overwrite the data the user entered on tab 1. The user might spot this by looking closely at the tag table on tab 2, but then he may not. This non-obvious behaviour was removed.
  • The exception thrown when closing an already closed changeset was fixed.
  • More cosmetic changes to the dialog.
  • Maybe also a solution to #19319, #21387 (added revalidate()).

comment:3 by Don-vip, 3 years ago

Thank you!

comment:4 by gaben, 3 years ago

Something is still not right with the upload dialog. Now I ran into an issue when an open changeset doesn't show up at all but a closed one does.

Edit: It never worked correctly, but at least reopening the upload dialog / clicking the refresh button fixed the issue, until now.

Last edited 3 years ago by gaben (previous) (diff)

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.