Signal-android: Show an inline map instead of a Google Maps link

Created on 11 Mar 2017  路  11Comments  路  Source: signalapp/Signal-Android

I have:


It'd be nice if, instead of showing a Google Maps link, Signal displayed a map inline. This would improve the UX for location "attachments" quite a bit, IMO.

Most helpful comment

@strugee There is a strong privacy reason for sending an image. Sending an "interactive map" means sending a location and having Signal (or an external library) query a server to load the map, leaking the exchanged location.

You could think of a "load location" button that prevents the map from auto-loading, but this is basically how it works now, where the static image is the button.

All 11 comments

OpenStreetMap would be an ideal match for most locations

Related: #5042

It'd be nice if, instead of showing a Google Maps link, Signal displayed a map inline.

I'm sorry if I don't understand your request correctly, but doesn't Signal already do both (i.e. send a link together with a map)?

@haffenloher no, I'm pretty sure it doesn't - that's the point of this issue. I'll double-check though.

Huh, looks like I was wrong. For some reason I thought this didn't work.

Though on second thought, this seems to just be an image. It'd be nice if the map was actually interactive and let you open a map view by tapping on it, without actually leaving the app.

@strugee There is a strong privacy reason for sending an image. Sending an "interactive map" means sending a location and having Signal (or an external library) query a server to load the map, leaking the exchanged location.

You could think of a "load location" button that prevents the map from auto-loading, but this is basically how it works now, where the static image is the button.

@paride oh, that's a great point. So I see two alternatives here:

  1. Keep the existing behavior
  2. Use offline OpenStreetMap tiles, if available

I'll let Moxie or one of the other developers make the final call, but I'm betting on option 1.

This was the commit message of the commit which introduced location sharing:

Start with encoding as a simple image thumbnail for compatibility with MMS and iOS.

Source: bcf95e50aa38cae2dfe640ea1a1e26cbaa3a84b7
So I would say there's something on the roadmap.

I think the idea is to sync up Android and iOS's attachment capability and then add more to both.

GitHub Issue Cleanup:
See #7598 for more information.

Was this page helpful?
0 / 5 - 0 ratings