#10812 closed enhancement (fixed)
[patch] Geo Tagged Images popup should show path
Reported by: | amenk | Owned by: | amenk |
---|---|---|---|
Priority: | normal | Milestone: | 14.12 |
Component: | Core image mapping | Version: | tested |
Keywords: | Cc: |
Description
When loading lots pictures with identical names but different paths we can not determine the on-disk path of a picture from the popup dialog.
The path should be shown somewhere or a function offered to copy the full path + name
Attachments (2)
Change History (10)
comment:1 by , 10 years ago
comment:2 by , 10 years ago
Milestone: | → 14.12 |
---|---|
Summary: | Geo Tagged Images popup should show path → [patch] Geo Tagged Images popup should show path |
Thanks for the patch, it will be applied after the current release.
Please read your patch carefully before submitting and make sure, it doesn't include any unrelated changes. The * import should be expanded.
comment:3 by , 10 years ago
Owner: | changed from | to
---|---|
Status: | new → needinfo |
Thanks for the patch. As said it requires some cleaning:
- MainApplication.java and TMSLayer.java must not be modified
- the import java.awt.datatransfer.* must be expanded
- the indentation must be OK, with spaces only
- there's a big problem with the code copied&pasted as it modifies the delete button, not the new button you want to add:
btnDeleteFromDisk.getInputMap(JComponent.WHEN_IN_FOCUSED_WINDOW).put(scCopyPath.getKeyStroke(), ACOPYPATH); btnDeleteFromDisk.getActionMap().put(ADELFROMDISK, copyPathAction);
comment:6 by , 10 years ago
Thanks for the patch!
I've removed the shortcut as I think it is not needed and might conflict with the ability to copy text elsewhere in the application.
comment:8 by , 10 years ago
Okay I didn't read it properly. As the suggested shortcut is Ctrl+Alt+Shift+C, this should be fine.
Another useful (similar) feature would be to add the image path to the "source:ref" tag (with a prefix) - as we would upload the images to our webserver.