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

Fix/add pip config ios #2562

Closed
wants to merge 3 commits into from

Conversation

g7i
Copy link

@g7i g7i commented Dec 20, 2021

Describe the changes

Add audio configs required for PIP for iOS

@g7i
Copy link
Author

g7i commented Dec 21, 2021

fixes #2561

@keyberth-rengel
Copy link

how's it going? Hey, how could you do picture-in-picture? I've been trying for many months but I haven't found any information. Could you help me?

Copy link
Contributor

@mattleff mattleff left a comment

Choose a reason for hiding this comment

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

This change should be guarded by an iOS version check

// Create new controller passing reference to the AVPlayerLayer
_pipController = [[AVPictureInPictureController alloc] initWithPlayerLayer:_playerLayer];
_pipController.canStartPictureInPictureAutomaticallyFromInline = true;
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
_pipController.canStartPictureInPictureAutomaticallyFromInline = true;
if (@available(iOS 14.2, *)) {
_pipController.canStartPictureInPictureAutomaticallyFromInline = true;
}

@hueniverse
Copy link
Contributor

@g7i Thank you for the PR. Can you review @mattleff suggestion? Also, how can we test this change?

@hueniverse
Copy link
Contributor

Closing due to lack of response (from the original submitted or anyone else to review and rebase). If someone wants to pick this up please open a new PR that ports this code into the current master. We are doing our best to start clean with v6 which requires being a bit aggressive in closing stale PRs that are over a year old. Please don't consider this dismissing of your work and contribution - it's greatly appreciated.

@hueniverse hueniverse closed this Jun 23, 2022
@hueniverse hueniverse added the stale Closed due to inactivity or lack or resources label Jun 23, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Platform: iOS stale Closed due to inactivity or lack or resources
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants