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

BUG: Minimize the video screen on IOS. #13

Open
viktor-shmigol opened this issue Oct 25, 2021 · 5 comments
Open

BUG: Minimize the video screen on IOS. #13

viktor-shmigol opened this issue Oct 25, 2021 · 5 comments
Labels
bug Something isn't working

Comments

@viktor-shmigol
Copy link
Contributor

Minimize feature is working on Android.
However, it's not working on IOS. For instance: if you click on minimize button it will not minimize the app.

Selection_023

@Zeno97
Copy link
Owner

Zeno97 commented Oct 27, 2021

Hi. I've read the documentation of the official jitsi meet sdk for ios about picture-in-picture
https://jitsi.github.io/handbook/docs/dev-guide/dev-guide-ios-sdk#picture-in-picture

In the cordova implementation we need to manage ourselves the view resizing.
Sdk for android instead manage pip by itself, i thought the same was for ios.

Asap i will work on this.

For the next week a fix will be released.
Thank you for your interest.

@Zeno97 Zeno97 added the bug Something isn't working label Oct 27, 2021
@Zeno97
Copy link
Owner

Zeno97 commented Oct 31, 2021

Little update:

Actually I am working on pip mode for ios but it seems will take longer.
I've found two solutions.
First is translating this solution from official jitsi from swift to objective c
https://github.com/jitsi/jitsi-meet/blob/master/ios/sdk/src/picture-in-picture/PiPViewCoordinator.swift

Second is implementing this framework in the plugin
https://github.com/Kofktu/PIPKit

As told in my last message, jitsi meet sdk require the developer to implement his own NATIVE implementation for PIP.
Jitsi Meet SDK doesn't use Picture in Picture capability.

@youssmak
Copy link

hi @Zeno97, any update ?

@Zeno97
Copy link
Owner

Zeno97 commented Nov 19, 2021

In this period I'm busy with work. If you need it ASAP you should fork and try to solve it yourself following my analysis in the last comment

@youssmak
Copy link

thks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants