Modify

Opened 15 years ago

Closed 15 years ago

Last modified 15 years ago

#3791 closed enhancement (wontfix)

[PATCH] nicer upload dialog

Reported by: andre68@… Owned by: team
Priority: trivial Milestone:
Component: Core Version: latest
Keywords: upload gui Cc:

Description

I find the current upload dialog a bit confusing so I decided to do my first development on JOSM. The patch rearranges some parts of the upload dialog. No functionality changes so far.

I've also attached a screenshot of the new dialog so you can have a look without applying the patch.

Any comments welcome.

What I'd like to see in the future is, that if the upload is done with an existing changeset the comment of the changeset is taken and not the comment of the upload dialog. If someone wants to change the comment he can do this on the changeset tab. Reason: When multiple changesets are open it is confusing if you have to change both the changeset and the comment before uploading. If you forget to change the comment in the dialog the comment of the changeset is changed silently.

Andre

Attachments (3)

new-upload-dialog.diff (4.1 KB ) - added by andre68@… 15 years ago.
The upload ui patch
new-upload-dialog.png (37.3 KB ) - added by andre68@… 15 years ago.
The screenshot of the new upload ui
old-upload-dialog.png (26.5 KB ) - added by andre68@… 15 years ago.
Old upload dialog box with invisible changes

Download all attachments as: .zip

Change History (7)

by andre68@…, 15 years ago

Attachment: new-upload-dialog.diff added

The upload ui patch

by andre68@…, 15 years ago

Attachment: new-upload-dialog.png added

The screenshot of the new upload ui

comment:1 by stoecker, 15 years ago

A solution would be: Preselect current comment for open changesets instead of last one.

comment:2 by Gubaer, 15 years ago

Resolution: wontfix
Status: newclosed

I'd rather not apply your patch. The suggested new layout isn't better than the current one. In particular it suggest that an upload comment can be entered for a new changeset only, which is not the case: it should always be entered. If you upload to an existing changeset it should be possible to update the update comment.

comment:3 by andre68@…, 15 years ago

Well, as I said the patch is an enhancement and I think that some more changes need to be done.

The current dialog is not good since you can not see the comment for the open changesets unless you make the dialog very wide which causes the layout manager to rearrange the upper part of the dialog so that in some cases (with some window heights) you cannot see, what changes you are going to upload. Will attach a screenshot of a bad window here so that you can have a look.

And as I already said I find it very confusing changing both the changeset and the comment if I want to upload something and have multiple open changesets. Thus, I think that the comment should be entered when opening the changeset with the ability to change the comment on the changeset tab. The current behaviour suggests that the user can change the comment for each change which is not the case. Only one comment for the changeset is possible.

Next step for this purpose would be to disable the comment line if an open changeset is used.

Will shift this discussion to the mailing list this afternoon to hear what other developers/users think about it...

by andre68@…, 15 years ago

Attachment: old-upload-dialog.png added

Old upload dialog box with invisible changes

comment:4 by avarab@…, 15 years ago

For what it's worth I've uploaded at least 3 changesets with the wrong comment=* because I was managing multiple open changesets at once and the design of the old upload dialog was such that when fiddling with it I managed to accidentally overwrite the comment on the server with another local comment.

This new dialog makes a much better use of available space. However existing upload comments should also be easy to change, just not so easy that one can do it accidentally.

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.