Skip to content
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

Video in gallery shows spinner rather that using thumbnails, so loads slowly #3771

Open
Tracked by #3777
ara4n opened this issue Feb 8, 2025 · 2 comments
Open
Tracked by #3777
Labels
A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect

Comments

@ara4n
Copy link
Member

ara4n commented Feb 8, 2025

Steps to reproduce

  1. open gallery from room settings
  2. open media
  3. swipe around until you hit a video which you don’t have cached locally
  4. stare at a spinner for ages while it loads

Outcome

What did you expect?

use the thumbnail to show you a placeholder so you immediately know what the gallery item is, and then perhaps only load the video if the user taps on it.

alternatively, stream the decryption so you rapidly get a preview inage from the first frame of the full res vid.

What happened instead?

stare at spinner for ages, wondering if it’s crashed, and what it’s trying to show you, and why it didn’t use the thumbnail.

also, chews bandwidth downloading a vid you might not care about.

Your phone model

No response

Operating system version

No response

Application version

834

Homeserver

No response

Will you send logs?

No

@ara4n ara4n added the T-Defect label Feb 8, 2025
@ara4n
Copy link
Member Author

ara4n commented Feb 8, 2025

see also #3760

@pixlwave
Copy link
Member

pixlwave commented Feb 10, 2025

Added a little bit more context in #3777. I would imagine a more beneficial change here would be to first implement progress on media downloads so that we have some way to visualise how long it will take for any kind of large file to be previewable - ah although that doesn't help with knowing what you're downloading.

@pixlwave pixlwave added A-Media S-Minor Impairs non-critical functionality or suitable workarounds exist O-Occasional Affects or can be seen by some users regularly or most users rarely labels Feb 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media O-Occasional Affects or can be seen by some users regularly or most users rarely S-Minor Impairs non-critical functionality or suitable workarounds exist T-Defect
Projects
None yet
Development

No branches or pull requests

2 participants