Skip to content

GIF animations autoplay in chat messages#5969

Merged
mahibi merged 3 commits intonextcloud:masterfrom
anakin78z:autoplay-gifs
Mar 23, 2026
Merged

GIF animations autoplay in chat messages#5969
mahibi merged 3 commits intonextcloud:masterfrom
anakin78z:autoplay-gifs

Conversation

@anakin78z
Copy link
Contributor

Fixes #3975

🖼️ Screenshots

Screen_Recording_20260316_083132_Talk_1.mp4

🚧 TODO

  • ...

🏁 Checklist

  • ⛑️ Tests (unit and/or integration) are included or not needed
  • 🔖 Capability is checked or not needed
  • 🔙 Backport requests are created or not needed: /backport to stable-xx.x
  • 📅 Milestone is set
  • 🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

Hide filename under GIF images.

Signed-off-by: Jens Zalzala <jens@shakingearthdigital.com>
@AndyScherzinger AndyScherzinger added enhancement New feature or request 3. to review Waiting for reviews labels Mar 17, 2026
Copy link
Collaborator

@mahibi mahibi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @anakin78z
tested and works fine 👍

It can be merged,
just you are aware: half parts of this code might never be released as is because it will be done differently in #5635
With Compose it might also be easier to add rounded corners...

@nickvergessen
Copy link
Member

Should respect
https://github.com/nextcloud/spreed/blob/1273e41ad3bd3d04388a31b1f18593cc8bb83e64/docs/capabilities.md?plain=1#L65
if possible

@anakin78z
Copy link
Contributor Author

@nickvergessen I've added a new commit to check the max-gif-size before autoplaying GIF files.

@mahibi
Copy link
Collaborator

mahibi commented Mar 23, 2026

@nickvergessen I've added a new commit to check the max-gif-size before autoplaying GIF files.

thank you @anakin78z 👍
can you sign off the commit?

Also, would be nice to resolve the codacy issues: https://app.codacy.com/gh/nextcloud/talk-android/pull-requests/5969

Autoplay GIF file if within max GIF size.

Signed-off-by: Jens Zalzala <jens@shakingearthdigital.com>
Removed unused import statements

Signed-off-by: Jens Zalzala <jens@shakingearthdigital.com>
@anakin78z
Copy link
Contributor Author

@mahibi done & done. Codacy returns ✅ now.

@mahibi mahibi merged commit cc52fac into nextcloud:master Mar 23, 2026
13 of 17 checks passed
@AndyScherzinger AndyScherzinger added this to the 23.1.0 milestone Mar 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Animate GIFs

4 participants