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

Improve GIF playback speed on ProMotion devices #3299

Merged
merged 1 commit into from
May 5, 2022

Conversation

skylarbarrera
Copy link
Contributor

Bumps react-native-fast-image's SDWebImage dependency to include this change: SDWebImage/SDWebImage#3280

There's still an issue where if a GIF is in a sheet and you're dragging the sheet, the speed changes while dragging, but otherwise this fixes the issue

Fixes RNBW-2590

What changed (plus any additional context for devs)

fast image looks like its not being maintained anymore so we've patched and updated the SDWebImage to better handle promotion displays

PoW (screenshots / screen recordings)

Dev checklist for QA: what to test

Final checklist

  • Assigned individual reviewers?
  • Added labels?
  • Added e2e tests? if not please specify why
  • If you added new files, did you update the CODEOWNERS file?

Bumps react-native-fast-image's SDWebImage dependency to include this change: SDWebImage/SDWebImage#3280

There's still an issue where if a GIF is in a sheet and you're dragging the sheet, the speed changes while dragging, but otherwise this fixes the issue
@skylarbarrera skylarbarrera added the needs dev review Includes code review AND testing out the branch label May 4, 2022
@linear
Copy link

linear bot commented May 4, 2022

RNBW-2590 animated NFTs play too fast

User alextoma.eth
iPhone 13 pro max
iOS 15.2 (will update to 15.3.1)

User reports that animated NFTs are all moving too fast in their wallet.

Christian had reported something similar 2 weeks ago on testflight: https://rainbowhaus.slack.com/archives/C02C2FVC6N6/p1643863488458799

Copy link
Member

@brunobar79 brunobar79 left a comment

Choose a reason for hiding this comment

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

🚀

@skylarbarrera skylarbarrera merged commit d79788a into develop May 5, 2022
@skylarbarrera skylarbarrera deleted the @skylar/sdimage branch May 5, 2022 18:00
@skylarbarrera skylarbarrera restored the @skylar/sdimage branch May 5, 2022 20:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs dev review Includes code review AND testing out the branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants