paper-plane icon indicating copy to clipboard operation
paper-plane copied to clipboard

Support showing location messages

Open marhkb opened this issue 2 years ago • 5 comments

Bildschirmfoto vom 2022-06-12 23-45-11

marhkb avatar Jun 12 '22 21:06 marhkb

Now with avatar map markers for live locations.

Bildschirmfoto vom 2022-06-14 14-38-25

marhkb avatar Jun 14 '22 12:06 marhkb

After trying this is my touchscreen laptop I've realized that it's not a good idea to make the map navigable, because it interferes with the chat history scrolling while using touch. I think that the MessageMap widget should just be a sort of preview of the position, which can then be clicked to open a full map that can be navigated. Since the last bit it's a bit more involved, I suggest to just focus on a MessageMap for this PR.

melix99 avatar Jul 28 '22 13:07 melix99

The map is now insensitive. Besides, the OSM copyright text has been adapted to the new AdwAboutDialog.

marhkb avatar Jul 29 '22 21:07 marhkb

Just wondering if it could be added support for Venues too (Venues are almost the same as locations, but they come with a place name too instead of just an arbitrary location): Telegram Android Telegrand

alissonlauffer avatar Aug 12 '22 04:08 alissonlauffer

Just wondering if it could be added support for Venues too (Venues are almost the same as locations, but they come with a place name too instead of just an arbitrary location): Telegram Android Telegrand

Yes, I think this would be quite easy. But I would rather do that in a separate PR before this becomes too big.

marhkb avatar Sep 11 '22 09:09 marhkb

I've marked this as draft, it needs to be ported to the new MessageBubble widget. I've set the milestone for 0.2 for now, but I might change it back to 0.1 if it gets ported in time.

melix99 avatar Mar 22 '23 21:03 melix99