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

Animate avatars on hover #5922

Closed
turt2live opened this issue Jan 4, 2018 · 7 comments
Closed

Animate avatars on hover #5922

turt2live opened this issue Jan 4, 2018 · 7 comments
Labels
A-Avatar O-Uncommon Most users are unlikely to come across this or unexpected workflow T-Enhancement

Comments

@turt2live
Copy link
Member

Like Discord

@turt2live
Copy link
Member Author

"On the discord desktop program, when you hover over a users message, if their avatar is a gif, it will play."

https://matrix.to/#/!DgvjtOljKujDBrxyHk:matrix.org/$15151283293464906msfqu:matrix.org

@specing
Copy link

specing commented Jan 5, 2018

Can the client be made more efficient as far as processor utilization goes instead of adding features that are of no real use and only slow things down?

@turt2live
Copy link
Member Author

@specing this is just a feature request from someone, not a guarantee it'll be implemented tomorrow. Performance is an ongoing battle.

@ragnarov
Copy link

ragnarov commented Jan 6, 2018

This is the most useful feature.
If one finds it less useful then that doesn't mean that others find it useless too.

turt2live : Thank you for acknowledging such an useful feature. :)

@turt2live
Copy link
Member Author

As a point of information: riot uses server-side thumbnails to render the avatars and relies on those. Due to a bug in the media repo project I'm working on, it didn't thumbnail gifs and instead returned the source gif, causing riot to play the image.

animated avatars

So in reality this is probably a lot easier than it seems: Synapse could be made to resize the gif (through a fourth query param or automatically) and riot just needs to allow the image to play on hover (to avoid the screenshot I've posted here).

@SethFalco
Copy link

SethFalco commented Aug 24, 2022

I think this is the same as #18442?

Maybe one of them should be closed? (I'd favor this one since the other issue has more details.)

@SimonBrandner SimonBrandner added O-Uncommon Most users are unlikely to come across this or unexpected workflow and removed P3 labels Aug 24, 2022
@t3chguy
Copy link
Member

t3chguy commented Aug 24, 2022

Closing in favour of #18442

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-Avatar O-Uncommon Most users are unlikely to come across this or unexpected workflow T-Enhancement
Projects
None yet
Development

No branches or pull requests

8 participants