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

Support the runloop mode control for macOS. Which can be useful when user want to pause animation when drag the mouse, or presenting modal window #2886

Merged
merged 2 commits into from Nov 6, 2019

Conversation

dreampiggy
Copy link
Contributor

New Pull Request Checklist

  • I have read and understood the CONTRIBUTING guide

  • I have read the Documentation

  • I have searched for a similar pull request in the project and found none

  • I have updated this branch with the latest master to avoid conflicts (via merge from master or rebase)

  • I have added the required tests to prove the fix/feature I am adding

  • I have updated the documentation (if necessary)

  • I have run the tests and they pass

  • I have run the lint and it passes (pod lib lint)

This merge request fixes / refers to the following issues: ...

Pull Request Description

This runloop mode can use used for the use case above. We can do this better for our Mac SDWebImage user.

Demo video:

video2.mp4.zip

@codecov
Copy link

codecov bot commented Nov 5, 2019

Codecov Report

Merging #2886 into master will increase coverage by <.01%.
The diff coverage is 83.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #2886      +/-   ##
==========================================
+ Coverage    84.2%   84.21%   +<.01%     
==========================================
  Files          65       65              
  Lines        6409     6424      +15     
==========================================
+ Hits         5397     5410      +13     
- Misses       1012     1014       +2
Flag Coverage Δ
#ios 85.05% <88.88%> (-0.19%) ⬇️
#macos 83% <83.33%> (+0.16%) ⬆️
#tvos 85.19% <88.88%> (+0.08%) ⬆️
Impacted Files Coverage Δ
SDWebImage/Private/SDDisplayLink.m 89.47% <77.77%> (-0.39%) ⬇️
SDWebImage/Core/SDAnimatedImageView.m 77.93% <88.88%> (+0.86%) ⬆️
SDWebImage/Core/SDWebImageDownloaderOperation.m 92.4% <0%> (-0.92%) ⬇️
SDWebImage/Core/SDWebImagePrefetcher.m 89.93% <0%> (-0.63%) ⬇️
SDWebImage/Core/SDAnimatedImagePlayer.m 85.89% <0%> (+1.28%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 253a5eb...ed9696f. Read the comment docs.

…user want to pause animation when drag the mouse, or open the Save/Open modal window
…lback at all, match iOS/tvOS/watchOS behavior.
@dreampiggy dreampiggy added this to the 5.3.0 milestone Nov 6, 2019
@dreampiggy dreampiggy added the macOS AppKit for macOS label Nov 6, 2019
@dreampiggy dreampiggy merged commit 387944b into SDWebImage:master Nov 6, 2019
@dreampiggy dreampiggy deleted the feature_runloop_mac branch November 6, 2019 12:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
animated image macOS AppKit for macOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants