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

adding pip #51

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from
Open

adding pip #51

wants to merge 2 commits into from

Conversation

m-derakhshan
Copy link
Contributor

Hello.
As mentioned in Issue #19, and based on our needs, I tried to add pip to the player while the user minimizes the app and the video is playing. However, this integration is not complete, but you may consider it a starting point.

@m-derakhshan
Copy link
Contributor Author

m-derakhshan commented Nov 12, 2024

Please consider these points:

  • as commented in the code, it should integrate with core and pip configuration, so if the user adds pip configuration, it becomes available here. Make this if condition true based on that:
    027fed9#diff-99a58128817a3cbfdecb428f84d1c5c13e8a5e55d22eb6c06bcae7fdee912598R336

  • I moved the onPause codes into onStop cause when the user enters the pip mode, the pause is triggered, but we should keep the player playing. I couldn't think of a scenario where onPuase would be necessary over onStop, but if it is, we need to modify the pip helper so that when it goes into pip mode, it plays the video. (there is a method for this. I used to make the video on the full screen. It can be extended)

027fed9#diff-99a58128817a3cbfdecb428f84d1c5c13e8a5e55d22eb6c06bcae7fdee912598R552

  • Also, there is this issue shown in the video. I don't know if it is due to Pip or if it is full screen.
video_2024-11-12_19-50-01.mp4

@m-derakhshan
Copy link
Contributor Author

@ceyhun-o
Copy link
Contributor

Hi @m-derakhshan,

Thank you very much for your PR!

We will review this and make the necessary changes as soon as possible.

@mammad-derakhshan
Copy link

Good morning,
May I ask for a time estimate on the integration of PiP in the player?
We have an active project, and due to the absence of this feature, we are currently using a custom implementation. Our next release is scheduled for the beginning of January. Should we plan to continue with our custom implementation, or can we expect the built-in PiP feature by then? @ceyhun-o @MattiasBuelens
thank you

@ceyhun-o
Copy link
Contributor

ceyhun-o commented Dec 3, 2024

Hi @mammad-derakhshan,

I will let you know once we have the planning in place if we can implement this before beginning January.

@mammad-derakhshan
Copy link

mammad-derakhshan commented Dec 3, 2024 via email

@ceyhun-o
Copy link
Contributor

Hi @m-derakhshan,

Unfortunately this PR won't be completed before new year. We will get back to it after the holidays.

@mammad-derakhshan
Copy link

mammad-derakhshan commented Dec 12, 2024 via email

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.

3 participants