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

android: Fix animated GIFs in lightbox, by getting Fresco version automatically #5608

Merged
merged 1 commit into from
Dec 14, 2022

Conversation

gnprice
Copy link
Member

@gnprice gnprice commented Dec 13, 2022

This follows upstream's instructions:

Note: the version listed above may not be updated in time. Please
check ReactAndroid/gradle.properties in the main repo to see which
fresco version is being used in a specific tagged version.

https://reactnative.dev/docs/0.67/image#gif-and-webp-support-on-android

except in an automated way. Happily this is even reasonably clean.

Fixes: #5607

@gnprice gnprice added a-Android upstream: RN Issues related to an issue in React Native P1 high-priority a-lightbox labels Dec 13, 2022
This was referenced Dec 13, 2022
@chrisbobbe
Copy link
Contributor

Huh, cool. LGTM, please merge at will! 🙂

…omatically

This follows upstream's instructions:

  Note: the version listed above may not be updated in time. Please
  check `ReactAndroid/gradle.properties` in the main repo to see which
  fresco version is being used in a specific tagged version.

  https://reactnative.dev/docs/0.67/image#gif-and-webp-support-on-android

except in an automated way.  Happily this is even reasonably clean.

Fixes: zulip#5607
@gnprice gnprice merged commit b301e9d into zulip:main Dec 14, 2022
@gnprice gnprice deleted the pr-android-anigifs branch December 14, 2022 19:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a-Android a-lightbox P1 high-priority upstream: RN Issues related to an issue in React Native
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Animated GIFs broken in lightbox again, on Android
2 participants