Bug 458537

Summary:	Feature Request (KGeoTag): Import Google Maps location - Copy location from clipboard
Product:	[Applications] kgeotag	Reporter:	BOF <bugs_kde_org.5.kuru>
Component:	General	Assignee:	Tobias Leupold <tl>
Status:	RESOLVED FIXED
Severity:	normal	CC:	tl
Priority:	NOR
Version First Reported In:	1.2.0
Target Milestone:	---
Platform:	Other
OS:	Linux
Latest Commit:		Version Fixed/Implemented In:
Sentry Crash Report:

Description BOF 2022-08-31 07:40:06 UTC

SUMMARY
The core of this feature request is better compatibility with location data from Google Maps. The feature request requests to fix a few issues for images that have no or wrong GPS data stored in them and are replaced with Google Maps.

THE GOAL (aka 'What I want to achieve')
Sometimes I have images that have no or wrong GPS info stored in them. I check the location on Google Maps and copy the position from there. I mainly do that because it's very easy and convinient. The main problem is that the location copied from Google Maps can not used directly.
The goal of this feature request is that the location data from Google Maps can be used more easy / directly from the clipboard.

EXAMPLE
Let's say I want to add the location of the Leaning Tower of Pisa to one of my images.
The location on Google Maps would be 43.723011720295915, 10.39663301723743
I can right click the image in the list and manually enter the coordinates. I can enter or copy numbers into the box to set the location. I could also assign the location to the center of the map. This is a fast way but only if you can find your location on the map itself.

ISSUES
1. The position of location data is swap in Google Maps. So the first number in Google Maps is the second one in KGeoTag. If you just copy the numbers over your images will end up somewhere in Somalia/Africa.
2. The box for a lat/lon values only support 5 digits after the decimal point. If you copy longer numbers the number will just not be copied (without any note).
3. When you copy the location data from Google Maps the decimal point is a full point and the separator between the lat and lon values is a comma. When you want to copy eg. the lat value with a full point into the box, it does not work. When you replace the full point with a comma it does work. NOTE: Maybe it's because my system language is german where the comma usually is used as decimal point.

SUGGESTIONS
1. A "copy location from clipboard" entry would be good. Then you could right click+copy the location in Google Maps and then right click+copy location from clipboard in KGeoTag. A function behind this menu entry would check if it's a known location format and if so add it to the image's metadata.
2. This could also be done in form of a pop-up widow that is less strict about it's entries. If a value can not be accepted there should be a feedback note. It took quite some time to find out why KGeoTag did not copy certain locations.

SOFTWARE/OS VERSIONS
Operating System: KDE neon 5.25
KDE Plasma Version: 5.25.4
KDE Frameworks Version: 5.97.0
Qt Version: 5.15.5
Kernel Version: 5.15.0-46-generic (64-bit)
Graphics Platform: X11
Processors: 16 × AMD Ryzen 7 4800H with Radeon Graphics
Memory: 30.8 GiB of RAM
Graphics Processor: AMD RENOIR
Manufacturer: Micro-Star International Co., Ltd.
Product Name: Bravo 17 A4DDR
System Version: REV:1.0

Comment 1 Tobias Leupold 2022-08-31 21:11:12 UTC

Thanks for your feature request!

This shouldn't be too hard to implement. I'll have a look at it shortly.

Stay tuned ;-)

Comment 2 Tobias Leupold 2022-09-01 13:55:39 UTC

Git commit 9db53c50994629173712adadcbeb99e1c2d1ca7f by Tobias Leupold.
Committed on 01/09/2022 at 13:55.
Pushed by tleupold into branch 'master'.

Added setting corrdinates from the clipboard

M  +3    -0    ChangeLog.rst
M  +51   -0    src/ImagesListView.cpp
M  +3    -0    src/ImagesListView.h
M  +9    -0    src/MainWindow.cpp
M  +1    -0    src/MainWindow.h

https://invent.kde.org/graphics/kgeotag/commit/9db53c50994629173712adadcbeb99e1c2d1ca7f

Comment 3 Tobias Leupold 2022-09-01 13:57:30 UTC

Did you think of something like this?

You can now right-click an image or a selection of images and "Set coordinates from clipboard". As of now, Google's format is parsed, as well as OpenStreetMap's Geo-URI scheme.

Comment 4 BOF 2022-09-01 14:43:10 UTC

(In reply to Tobias Leupold from comment #3)
> Did you think of something like this?
> 
> You can now right-click an image or a selection of images and "Set
> coordinates from clipboard". As of now, Google's format is parsed, as well
> as OpenStreetMap's Geo-URI scheme.

Sounds pretty good.
I looked at the commit on gitlab and (saying this with the a rather small cpp code knowledge) it looks as if it would do exactly what I would use it for. Is there a place where I can download an AppImage to test if it works as designed?

PS: Thank you very much for the quick response!

Comment 5 Tobias Leupold 2022-09-01 14:52:24 UTC

There are no live builds. But it's pretty easy to compile KGeoTag, just have a look at https://community.kde.org/KGeoTag

Comment 6 Tobias Leupold 2022-09-04 12:32:53 UTC

I'm pretty sure this is what you wanted. I wanted to do a new release anyway, so I did. Just checkout KGeoTag 1.3.0 as soon as your distribution provides it.

If there's still a problem, feel free to re-open this bus or to file a new one.