Modify

Opened 9 years ago

Closed 6 years ago

Last modified 11 months ago

#12669 closed enhancement (fixed)

Disable upload and save buttons when not needed

Reported by: openstreetmap.org-user-d1g Owned by: Don-vip
Priority: normal Milestone: 19.10
Component: Core Version:
Keywords: disableditems upload save Cc: Klumbumbus

Description

No point in showing "No changes to upload" error.

If there no data in layer or if command stack is empty, then simply suppress 0 upload.

Attachments (0)

Change History (11)

comment:1 by Klumbumbus, 9 years ago

Cc: Klumbumbus added

comment:2 by openstreetmap.org-user-d1g, 9 years ago

Keywords: disableditems added

comment:3 by Don-vip, 6 years ago

Keywords: upload save added
Priority: trivialnormal
Summary: Disable "upload" button - similar to undo/redoDisable upload and save buttons when not needed

comment:4 by Don-vip, 6 years ago

Owner: changed from team to Don-vip
Status: newassigned

comment:5 by Don-vip, 6 years ago

Milestone: 19.10

comment:6 by Don-vip, 6 years ago

Resolution: fixed
Status: assignedclosed

In 15404/josm:

fix #12669 - Disable upload and save buttons when not needed

comment:7 by Klumbumbus, 6 years ago

Thats great! I'm just wondering in which situation the save button is disabled?

comment:8 by Don-vip, 6 years ago

When the .osm layer is associated to a file which has not been modified. So right after you load an existing .osm file, or right after you save .osm data to an osm file. If you find some situations were it does not behave correctly, let me know.

Last edited 6 years ago by Don-vip (previous) (diff)

in reply to:  8 comment:9 by Klumbumbus, 6 years ago

Replying to Don-vip:

When the .osm layer is associated to a file which has not been modified. So right after you load an existing .osm file, or right after you save .osm data to an osm file.

Indeed.

If you find some situations were it does not behave correctly, let me know.

Hm it seems to be a bit fishy when doing undo redo. I faced different strange situation but was not able to properly reproduce them.

comment:10 by Don-vip, 6 years ago

I made a few undo/redo tests before committing and noticed no problem. But I only tested common cases (node movement, object creation/deletion, tag deletion).

comment:11 by GerdP, 11 months ago

See #23648 regarding the disabled Save action

Modify Ticket

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