You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I'm rendering several PINAnimatedImageView in a UIScrollView. As the user scrolls, it sets isPlaybackPaused to true when shown on screen and false when off screen. Overall, I really like this library. The performance of my app is meaningfully better than the other two libraries I integrated before this.
There are three UX issues that I'd like to report.
1. When PINAnimatedImageView is initially paused, it does not display the cover image
When/how is the cover image displayed?
I've noticed that whenever isPlaybackPaused is set to true before the image loads, the coverImage is never displayed, so the user sees an empty UIView and its sad.
Expected behavior: Even while paused, render the cover image once the image loads.
2. Pausing playback makes the PINAnimatedImageView display nothing
Currently, when isPlaybackPaused is false, the PINAnimatedImageView displays nothing. If there's any delay between pausing and unpausing (such as, the JS bridge when using this library in a React Native app), there are brief moments of black nothingness where the most recently played frame is supposed to go.
Expected behavior: Display the last image rendered before the animation was paused (maybe not setting frameImage to nil would work?)
I spent a couple of hours investigating this today. I think the image frame is being released here:
stopAnimating, (which isPlaybackPaused calls) causes the PINCachedAnimatedImage to start over from the first frame the next time startAnimating is called. This appears to be a deliberate implementation detail. Is resume a thing other people care about too or is it just me?
Its quite likely that 1 and 2 are the same issue.
The text was updated successfully, but these errors were encountered:
I'm rendering several
PINAnimatedImageView
in aUIScrollView
. As the user scrolls, it setsisPlaybackPaused
totrue
when shown on screen andfalse
when off screen. Overall, I really like this library. The performance of my app is meaningfully better than the other two libraries I integrated before this.There are three UX issues that I'd like to report.
1. When
PINAnimatedImageView
is initially paused, it does not display the cover imageWhen/how is the cover image displayed?
I've noticed that whenever
isPlaybackPaused
is set totrue
before the image loads, thecoverImage
is never displayed, so the user sees an empty UIView and its sad.Expected behavior: Even while paused, render the cover image once the image loads.
2. Pausing playback makes the
PINAnimatedImageView
display nothingCurrently, when
isPlaybackPaused
is false, thePINAnimatedImageView
displays nothing. If there's any delay between pausing and unpausing (such as, the JS bridge when using this library in a React Native app), there are brief moments of black nothingness where the most recently played frame is supposed to go.Expected behavior: Display the last image rendered before the animation was paused (maybe not setting
frameImage
tonil
would work?)I spent a couple of hours investigating this today. I think the image frame is being released here:
PINRemoteImage/Source/Classes/AnimatedImages/PINAnimatedImageView.m
Line 237 in 5e51b81
3.
isPlaybackPaused
stops instead of pausesstopAnimating
, (whichisPlaybackPaused
calls) causes thePINCachedAnimatedImage
to start over from the first frame the next timestartAnimating
is called. This appears to be a deliberate implementation detail. Is resume a thing other people care about too or is it just me?Its quite likely that 1 and 2 are the same issue.
The text was updated successfully, but these errors were encountered: