-
Notifications
You must be signed in to change notification settings - Fork 2
v3.6.1 #10
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
base: master
Are you sure you want to change the base?
v3.6.1 #10
Conversation
Co-authored-by: Mark Vayngrib <[email protected]>
Co-authored-by: Mark Vayngrib <[email protected]>
* feat: support video splashscreen on iOS * feat: support video splashscreen on Android * fix: responsiveness * docs: update readme with new api * refactor: minor cleanup * refactor: include video_view in this module Co-authored-by: Alexander Pataridze <[email protected]>
* feat: allow pausing video * feat: add option to remove pause callback & resume video * feat: add option to pause video * feat: add option to remove pause callback & resume video * fix: android hideVideo * refactor: minor cleanup Co-authored-by: Alexander Pataridze <[email protected]>
andrejborstnik
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@arienemaiara let's open a PR with your changes to master & merge that. Also cherry-pick to 3.X branch.
Let's also make looping an optional feature in the api
| + (void) restartVideoFromBeginning:(AVPlayerItem*)playerItem { | ||
| [lastPlayer seekToTime:kCMTimeZero]; | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to check if lastPlayer !== nil first
No description provided.