TeamNewPipe / NewPipe

A libre lightweight streaming front-end for Android.
https://newpipe.net
GNU General Public License v3.0
31.35k stars 3.05k forks source link

Add fullscreen default quality option #4392

Open TippyLion28 opened 4 years ago

TippyLion28 commented 4 years ago

Checklist

Describe the feature you want

Now that videos can be played within the information screen seperately from fullscreen, it makes sense to add an option to change the default fullscreen quality independently.

Is your feature request related to a problem? Please describe it

Additional context

How will you/everyone benefit from this feature?

Not really much I can say here other than I think this is a common sense request :-)

opusforlife2 commented 4 years ago

Hmmm. Not sure this can be done. The player doesn't change upon rotation.

TippyLion28 commented 4 years ago

Maybe keep the current behaviour but add the option to change quality on fullscreen? Same with popup player, quality change could be completely optional :)

opusforlife2 commented 4 years ago

That's not an issue with the popup player because it is different from the main player. Rotation, on the other hand, just modifies the main player's dimensions. Anyway, let some developer have a look at this issue. They would know better what to do.

Stypox commented 4 years ago

I think this would ruin the experience: the whole point of having a unified ui was to be able to switch seamlessly between players without having to reload the stream. For example, the stream is kept the same when switching from main to popup, if main was already playing. So changing the quality when pressing fullscreen would just be bad for the user in my opinion. Also note that the difference in bandwidth between 360p (the current default for popups) and 720p60 (the current default for main) is small.

opusforlife2 commented 4 years ago

the stream is kept the same when switching from main to popup, if main was already playing

Not if popup has a different default resolution. And even then, I don't think the code recognises that it is the same stream. There is still some delay between switching.

360p (the current default for popups)

480p.

TippyLion28 commented 3 years ago

Adding onto this, YouTube in the browser dynamically adjusts quality to match the size of the player when the quality is set to auto. When I fullscreen the player it switches to a higher quality automatically. It does this without any stuttering or delay.

Instead of eagerly blowing away the contents of the buffer, it just begins to buffer higher quality content in a lazy fashion. That way, when the lower quality buffer has been completely consumed, the higher quality video begins seamlessly.

opusforlife2 commented 3 years ago

@TippyLion28 Indeed. And for that we first need to implement DASH playback properly. That PR is in the works, and you can look for it.

TippyLion28 commented 3 years ago

Awesome, looking forward to that :)