talk-android icon indicating copy to clipboard operation
talk-android copied to clipboard

Better in-chat media preview

Open AndyXheli opened this issue 3 years ago • 12 comments

I don't know how hard it would take to implement but it sure would be nice.

Here's an in-chat image preview on Nextcloud Android Talk and the corresponding view on Telegram

Talk Screenshot_20220913-144506_Talk QA Telegram166503232-759961f1-37eb-42ce-af07-e93fb289b864

Similar too https://github.com/nextcloud/talk-ios/issues/776

AndyXheli avatar Sep 13 '22 19:09 AndyXheli

@andyxheli is the image in both screenshots the same one, since one seems to be a photo from a monitor image and with a different image ratio, no?

AndyScherzinger avatar Sep 13 '22 19:09 AndyScherzinger

Hi @AndyScherzinger what I mean is maybe removing the name of the photo from that chat and making it similar to telegram I'm sorry about that bad photos

Telegram Screenshot_20220913-150520_TelegramScreenshot_20220913-150605_Talk

AndyXheli avatar Sep 13 '22 20:09 AndyXheli

needs to be aligned with the talk iOS app. there we also have the filename under the photos. on web it's a mouseover tooltip.

if we decide to hide the filename then it would also make sense to move the time onto the photo like telegram does.. the filename would still be visible when opening the photo in fullscreen..

mahibi avatar Sep 14 '22 06:09 mahibi

I think this would be a good approach while currently we do not have a dedicated "image-bubble" but a preview one and this approach only works well for images but not for other files like pdfs, so we would need to add a distinction and a specific image bubble I think.

AndyScherzinger avatar Sep 14 '22 07:09 AndyScherzinger

Hey guys here's a screenshot of how it looks when you send photos with big file name. Personally it would be nice to remove the name of the photos from the viewer Screenshot_20220918-162726_Talk.jpg

AndyXheli avatar Sep 18 '22 21:09 AndyXheli

@jancborchardt @nimishavijay what do you think? https://github.com/nextcloud/talk-android/issues/2366#issuecomment-1245901961

I like it and agree with @mahibi - if we do it we need to do it i Android and iOS as well as for photos only think.

AndyScherzinger avatar Sep 19 '22 15:09 AndyScherzinger

Agreed, file names for images are usually technical and not useful, so it makes sense to remove it.

As for an "Image bubble" I think it's a good idea. Telegram and WhatsApp have a bubble, and whenever there is an image caption it is a part of the bubble. So if we are planning to have image captions in the future a bubble is certainly the right direction.

Also agreed that the timestamp can be within the image. Note that there is always a half transparent black background with the timestamp text in white (in WhatsApp, Telegram and Signal) so that the timestamp is visible if the image is white.

For other file types, the name is useful but I think it can still be within a bubble and the name ellipsized, so that the size still remains consistent. Similar to WhatsApp and telegram

Screenshot_20220919-174900_Telegram.jpg

Screenshot_20220919-174954_WhatsApp.jpg

nimishavijay avatar Sep 19 '22 15:09 nimishavijay

Ah, I see for the other file types, yes makes sense in case we would have a preview available. Not sure we know that upfront all the time though.

AndyScherzinger avatar Sep 19 '22 17:09 AndyScherzinger

Not implementing it until https://github.com/nextcloud/talk-android/issues/1552 is resolved

provokateurin avatar Sep 30 '22 13:09 provokateurin

I don't know about the overall plan, but #2366 (this issue) says the solution won't be implemented until #1552 is resolved. And #1552 says it won't be done until #2449 is merged.

With #2449 now merged, can we rethink #1552 and, eventually, #2366 (this issue), which I would very much like to see resolved?

Thanks!

louim-lbs avatar Sep 28 '23 14:09 louim-lbs

Hey I would agree any plans on getting this implemented?

AndyXheli avatar Jan 31 '24 14:01 AndyXheli

Well, all devs are currently busy with feature, of course contributions are aways welcomed, also for this one. Maybe somebody finds the time when limiting it to image previews for the momentum and how the issue discussion initially started.

AndyScherzinger avatar Jan 31 '24 22:01 AndyScherzinger