Skip to content

Fix for https://github.com/eduardoRoth/media-player/issues/72: Fix lo… #73

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

flexgrip
Copy link

Fixes #72 "Loading animation covers play button with mp3's on iOS"

  • Adds isAudioOnly method in MediaPlayerController.swift
  • Adds buffer checks for audio only playback scenarios and toggles loading in MediaPlayerControllerObservers.swift

@flexgrip
Copy link
Author

I probably should have asked if this was even relevant, since you mentioned you were close to being done with v5.0? Anyway, just wanted to chip in. Thanks!

Copy link
Owner

@eduardoRoth eduardoRoth left a comment

Choose a reason for hiding this comment

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

@flexgrip thank you for this PR!!! I appreciate all the help I can get on this plugin 😄
Just a minor change, could you extract this code to another function, like "setAudioOnlyObservers" or something like that? Just so we keep that function small and have this logic outside

@eduardoRoth
Copy link
Owner

@flexgrip BTW, sorry about the late reply, I've had a lot going and haven't had the time to work on this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Loading animation covers play button with mp3's on iOS
2 participants