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

Limit resolution to what what is actually displayed #1129

Closed
chaptergy opened this issue Mar 20, 2021 · 3 comments · Fixed by #4978
Closed

Limit resolution to what what is actually displayed #1129

chaptergy opened this issue Mar 20, 2021 · 3 comments · Fixed by #4978
Labels
enhancement New feature or request player-related Player-related changes are currently on hold due to the ongoing player migration.

Comments

@chaptergy
Copy link
Contributor

Is your feature request related to a usage problem (not a bug)? Please describe.
I was having the issue that some videos were buffering all the time, while most just work fine. I found, that FreeTube always tries to play at the highest resolution (as described in issue #595), so it was loading 4K video when I am watching on a 1080p screen not even in fullscreen.

Describe the solution you'd like to see implemented
Limit the resolution to what can actually be displayed on screen. so if the video on screen will show in an area of maybe 680x1000px, it should go no further than the 720p stream.

Describe alternatives you've considered
Add a maximum resolution to the settings. This way I can at least say to use 1080p max, since I have no screens with a greater resolution than full HD.

Additional context
Of course this would only be possible if actual control over the resolution is possible, but the comments in the issue linked above give me hope this is possible in the future.

@chaptergy chaptergy added the enhancement New feature or request label Mar 20, 2021
@GilgusMaximus
Copy link
Contributor

We deliberately deactivated that feature. Although it didn't stop displaying larger resolutions while it was deactivated either.

But it should be possible to add a settings option for that. Although fixing #595 will probably already be enough then

@ghost
Copy link

ghost commented Jul 6, 2021

Describe the solution you'd like to see implemented
Limit the resolution to what can actually be displayed on screen. so if the video on screen will show in an area of maybe 680x1000px, it should go no further than the 720p stream.

Describe alternatives you've considered
Add a maximum resolution to the settings. This way I can at least say to use 1080p max, since I have no screens with a greater resolution than full HD.

You may well be creating another problem, whereby the software needs to reliably and consistently detect screen resolutions and I have multiple screens... How would that work on my set-up?

One is 1440x900 and the big one is 1920x1080p - What would your player do on my machine and how reliable would it need to be?? O_o

I think this would be a step in the wrong direction.

IMHO, it would be better (and more intuitive) to stick to displaying the available qualities in the video player quality list and honouring the Default Quality setting, which you can set for your main playback screen(s).

I would, however, like to see different resolution selections for small/page and full-screen players. Ie, I could choose 360p for windowed playback and 1080p for full-screen playback - Like the YouTube Enhancer plug-in ;)

Any thoughts? Have I completely missed the point?

@GilgusMaximus
Copy link
Contributor

You can get all screens via Electron an then just take the largest in terms of width.
This way it is safe to say that the app covers all possible resolutions available.

But as this has been stale and not really on any priority list, do not fear that this will come quickly. Additionally I also have a second monitor to work with

@kommunarr kommunarr added the player-related Player-related changes are currently on hold due to the ongoing player migration. label Apr 17, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request player-related Player-related changes are currently on hold due to the ongoing player migration.
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants