Skip to content

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Show albums for multiple images #753

Closed
turt2live opened this issue Dec 22, 2017 · 3 comments
Closed

Show albums for multiple images #753

turt2live opened this issue Dec 22, 2017 · 3 comments
Labels
A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely T-Enhancement X-Needs-Product More input needed from the Product team X-Spec-Changes

Comments

@turt2live
Copy link
Member

Description

This could either be a new event type or determined on the fly by Riot. The idea would be that if someone was uploading 2 or more images to the room, then the images get put into an album for scrolling through - much like Instagram. The download link should be for the entire set, with per-image download links being accessible somewhere (like the lightbox).

I'd recommend a new event type of m.album which would essentially be an array of m.image bodies. However, there's also the possibility of Riot automatically grouping images to albums using some rules:

  • Only group images if they are consecutive and from the same user
  • Don't group images if they were sent too far apart (~1min?) - group based on previous image
    • Slow uploads are the reason to look at the previous image. For example, if A was uploaded at 00:00:00, B at 00:00:45, and C at 00:01:10 - A, B, and C should be grouped, even though C is >1 min away from A (as it is <1 min away from B).

Version information

  • Platform: web (in-browser)
  • Browser: Chrome 63
  • OS: Windows 10
  • URL: riot.im/develop
@MurzNN
Copy link

MurzNN commented Jan 21, 2019

Better way is attach multiple images to single message in UI, so when user composing message text - Upload file button will upload attached file to current composing message (when it will be sent), instead of immediately send this file as separate message to room. This is like many other messengers works with image attachments.
After implementing this will be much easier to detect albums and make slideshow from images.
Here https://github.com/vector-im/riot-web/issues/6295 is my feature request about this.

@t3chguy
Copy link
Member

t3chguy commented Dec 23, 2019

Related to element-hq/element-web#337

@kittykat
Copy link
Contributor

Moving to a meta discussion as this is cross-platform

@kittykat kittykat transferred this issue from element-hq/element-web Oct 13, 2022
@element-hq element-hq locked and limited conversation to collaborators Oct 13, 2022
@kittykat kittykat converted this issue into discussion #754 Oct 13, 2022

This issue was moved to a discussion.

You can continue the conversation there. Go to discussion →

Labels
A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely T-Enhancement X-Needs-Product More input needed from the Product team X-Spec-Changes
Projects
None yet
Development

No branches or pull requests

7 participants