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

thumbnails switch to blurhash and back again when timeline reloads #3594

Open
ara4n opened this issue Dec 9, 2024 · 1 comment
Open

thumbnails switch to blurhash and back again when timeline reloads #3594

ara4n opened this issue Dec 9, 2024 · 1 comment
Labels
A-Media T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter

Comments

@ara4n
Copy link
Member

ara4n commented Dec 9, 2024

Steps to reproduce

  1. Open a room with images and start reading it
  2. Thumbnails are visible…
  3. …until the timeline reloads due to a SS refresh of some kind…
  4. and the thumbnails you were just looking at get replaced by blurhashes
  5. you then have to wait at least a second or so before the thumbnails unblur

Outcome

What did you expect?

given you were just looking at the thumbnail, you’d expect it to still be there, even if the timeline reloads, rather than it disappearing into a blur for a bit

Your phone model

No response

Operating system version

No response

Application version

783

Homeserver

No response

Will you send logs?

No

@pixlwave
Copy link
Member

pixlwave commented Dec 9, 2024

They're switching back to the unloaded state as the stable ID changes which is what the view's identity is tied to - we started using the stable ID rather than the upload URL to fix the issue where the local echo would flicker (#3525). That said, it sounds weird that it is taking a whole second to load, I would expect it to load immediately from the cache/store and simply flicker like the local echoes were doing.

Are you seeing this in both encrypted and unencrypted rooms?

@pixlwave pixlwave added A-Media T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter labels Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Media T-Defect X-Needs-Info This issue is blocked awaiting information from the reporter
Projects
None yet
Development

No branches or pull requests

2 participants