Picture in Picture mode for autoplay

[pablodekota]

Is your feature request related to a usage problem (not a bug)? Please describe.
When autoplay is active, the Picture in Picture mode doesnt automatically get activated on the new video, it just stays on the old one.

Describe the solution you'd like to see implemented
I would like the PIP mode to carry over on next videos when autoplay is on, tho i would still like it to stay when i click home, so i can find new videos while still watching the original one

Describe alternatives you've considered
I personally dont know how to do this, if i did, i would have made a PR, but as an example when you use firefoxes PIP mode, the next video uses PIP aswell.

[GilgusMaximus]

The PiP has some problems, or more like intended behavior. E.g. it does not go back to the video once you leave the page. To make it more useful a lot of work has to be done.

#624

[pablodekota]

The PiP has some problems, or more like intended behavior. E.g. it does not go back to the video once you leave the page. To make it more useful a lot of work has to be done.

#624

@GilgusMaximus
that is not my concern, that it not going back to a video can be overdone by just going back and minimizing it into the player and it will act as if you continued watching, my only real issue is that it wont go to the next video when autoplay is active

[SebTota]

Do you have any preference on how this should be implemented?

My idea is:

• Make sure the PiP window is destroyed when the current video finishes (this is probably something that should be added anyway when autoplay is on)
• Append a query parameter ?PiP=true to the recommended video URL (code reference)
• On player initialization, check for the query parameter and force a PiP window if true

We could also just keep it as a setting to keep track of if the last video had PiP mode on, but this just seemed like there would be less of a chance to keep PiP mode constantly on if something didn't go quite right.

If you are ok with my implementation just let me know and I can implement this.

[GilgusMaximus]

So this sounds like a good plan. But I'd maybe change one thing. Instead of adding the parameter to the URL, it'd be better to add a boolean value lastPlayedAsPip to the state of the app, so the information is stored across the videos for normal autoplay, YouTube playlists and custom playlists in the future. Because there is no format yet for custom playlists, and whether we'll actually store the whole URl or just the videoId, it'd be more future proof to decouple that from the next video.

But otherwise it sounds good

[efb4f5ff-1298-471a-8973-3d47447115dc]

closing in favor of #2295