Opened 2 years ago
Last modified 2 years ago
#22308 needinfo enhancement
[Patch] Add option to toggle layer read-only status to popup menu
| Reported by: | Woazboat | Owned by: | Woazboat |
|---|---|---|---|
| Priority: | normal | Milestone: | |
| Component: | Core | Version: | |
| Keywords: | layer, state, read-only, toggle, lock, locked | Cc: | MSiipola |
Description (last modified by )
Add a new menu item to the layer popup context menu that allows toggling of the read-only lock state for data layers
https://github.com/JOSM/josm/pull/99
https://patch-diff.githubusercontent.com/raw/JOSM/josm/pull/99.patch
Attachments (1)
Change History (7)
by , 2 years ago
| Attachment: | josm_layer_read_only_toggle.gif added |
|---|
comment:1 by , 2 years ago
| Description: | modified (diff) |
|---|
comment:2 by , 2 years ago
comment:3 by , 2 years ago
| Owner: | changed from to |
|---|---|
| Status: | new → needinfo |
@Woazboat: do you have any response for skyper's comment?
I'm going to hold off on merging this patch for now.
We'd probably have to have a secondary field (lockedOnLoad or something) to indicate that the provider of the data does not want the data to be accidentally uploaded. Either that, or if the dataset is locked on load, it cannot be unlocked via this popup menu option.
comment:5 by , 2 years ago
I would not ease the changing of locked and upload as they are usually set for good reason.
How about two more states, gui_locked and gui_upload, identical to locked and upload but changeable from the GUI.
comment:6 by , 2 years ago
| Cc: | added |
|---|---|
| Keywords: | state added |
-1
Modification is blocked by reason and I think this would make it to easy to change from
lockedstatus. An option to changing the status fordownload=*might be an improvement but not forlockedorupload.