sumo icon indicating copy to clipboard operation
sumo copied to clipboard

Published 20 hours ago • verified publisher icon eclipse-sumo

Add direct Google Maps and / or OSM shortcut to context menu

Open behrisch opened this issue 3 years ago • 10 comments

The whole copy cursor geo position thing still needs too many clicks. Maybe we can add a configurable URL template which the user can change via GUI settings

behrisch avatar Dec 14 '21 16:12 behrisch

Or even better, a shortcut to GeoHack (e.g. https://geohack.toolforge.org/geohack.php?params=52_31_17_N_13_24_37_E), just like Wikipedia does with coordinates.

This requires converting the coordinate to DMS.

angelobanse avatar Dec 15 '21 14:12 angelobanse

Good idea! And actually it does not require conversion, decimal degrees work just fine: https://geohack.toolforge.org/geohack.php?params=52.31;13.24

behrisch avatar Dec 15 '21 16:12 behrisch

@namdre @angelobanse Do we still want to have this configurable to save one more click?

behrisch avatar Dec 16 '21 20:12 behrisch

Yes

namdre avatar Dec 16 '21 20:12 namdre

And the configuration should persists in the fox registry. (I don't want it in a gui settings file since that might be a security issue).

namdre avatar Dec 16 '21 20:12 namdre

Should the configured service replace GeoHack - or would it be an additional option in the context menu?

angelobanse avatar Dec 16 '21 21:12 angelobanse

I think it should replace geohack with a menu entry "show curser geo-position in web map"

  • the Application Settings dialog could be used to configure the template string.
  • The default could be https://geohack.toolforge.org/geohack.php?params=LAT;LON
  • Calling the menu entry would replace LAT;LON with the respective values.
  • The user could then change the template to something like https://www.google.com/maps?ll=LAT,LON

namdre avatar Dec 16 '21 23:12 namdre

@angelobanse Could you take care of

  • [ ] make the table in the AppSettingsDialog more beautiful
  • [x] extend the list of defaults if useful, I currently use: https://www.google.com/maps?ll=%lat,%lon&t=h&z=18 and https://www.openstreetmap.org/?mlat=%lat&mlon=%lon&zoom=18&layers=M
  • [ ] document it
  • [ ] clean up leading empty rows when saving the table

Thank you!

behrisch avatar Mar 24 '22 08:03 behrisch

@angelobanse The name GoogleSat was there for a reason because it is actionally not only the map we are linking to but the hybrid map/satellite view. Maybe GoogleMapsSat? But I also don't mind if we keep the new name because it correctly names the provider.

behrisch avatar Oct 31 '23 11:10 behrisch

@behrisch let's keep it as "Google Maps". Satellite is only a view that can be changed with one click on the map.

angelobanse avatar Oct 31 '23 11:10 angelobanse