fix: Animate avatars/icons that's using the .webp file format

[pparaxan]

WebP can be animated, and so—it should have the same effect as normal GIFs—having the loop animation than it just being static.

[pparaxan]

DCO'd

[Taureon]

I wonder if it makes sense here to make a new Set so instead of adding a bunch of new conditions in the future we just do animatableContentTypes.includes(this.contentType)

[pparaxan]

I wonder if it makes sense here to make a new Set so instead of adding a bunch of new conditions in the future we just do animatableContentTypes.includes(this.contentType)

probably, but what other format are there that people uses on a regular meant for this case?

[Taureon]

I wonder if it makes sense here to make a new Set so instead of adding a bunch of new conditions in the future we just do animatableContentTypes.includes(this.contentType)

probably, but what other format are there that people uses on a regular meant for this case?

JXL?
That's one that comes to mind on the spot as I'm reading this notification

[Dadadah]

It looks like this should close stoatchat/for-web#827.

Were you able to test that this fix worked as expected @pparaxan ?

[pparaxan]

Were you able to test that this fix worked as expected @pparaxan?

If you're talking about if I made my own self-hosted instance with the patches and all and see if it works—no, I just use the context available on how the code works and impl the fix as needed.

afaik—there was no code related to this in the frontend, only here in the javascript client, in which—the frontend pulls as a Git submodule

[pparaxan]

It should work anyways because as says in the diff—that function is the one responsible for generating an URL of files.

[pparaxan]

afaik—there was no code related to this in the frontend, only here in the javascript client, in which—the frontend pulls as a Git submodule

I just tried it and it turns out somebody (in-between the time this PR was made and now) just made a fix over the frontend which I think eliminated the need for the /original query in general.

so I think if anything—you should just close this PR.

[Dadadah]

Closing.

FYI pparaxan you can choose to close PRs you opened yourself.

[pparaxan]

FYI pparaxan you can choose to close PRs you opened yourself.

Oh sorry, I just forgor that I made this PR. Should've closed it once I realised that it has been fixed.

edit: I don't use GitHub that much.

[Dadadah]

This isn't fixed. And this change does in fact get the webp's to animate so I'm gonna merge it

[Dadadah]

For future contributions, please ensure you run pnpm run fmt before opening a pr.