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] Bitrate selection menu pops up without any keyboard shortcut #8

Open
k939kk opened this issue Mar 22, 2024 · 7 comments
Open

Comments

@k939kk
Copy link

k939kk commented Mar 22, 2024

[bug] Inspite of set_max_bitrate set to true the bitrate selection menu keeps popping up on any new instance of netflix.

Also, it is a bit unclear what the override and reset buttons functions are in the bitrate selection menu. A small note in the readme would be useful.

Browser: Firefox 124.0.1

@DavidBuchanan314
Copy link
Owner

setting set_max_bitrate to true is what makes it pop up, set it to false if you want to invoke it manually

@DavidBuchanan314
Copy link
Owner

(the menu is supposed to disappear once the max bitrate has been selected, but it's buggy)

@borsigu
Copy link

borsigu commented Mar 22, 2024

is it possible to automatically pick the highest bitrate rather than picking from the pop-up every episode?

@DavidBuchanan314
Copy link
Owner

That's what set_max_bitrate is supposed to do, that's the option

@onli
Copy link

onli commented Mar 24, 2024

I found the reason, at least on my system.

set_max_bitrate() returns false when one of the selects is disabled. But the audio select is disabled for me. Thus I changed that abort:

to:

if (select.disabled){
	if (list == AUDIO_SELECT) {
		continue
	} else {
		return false;
	}
}

That wasn't enough though, because this check also failed:

if (options.length > 1 && options[0].selected == false){

I'm not sure what the intention was here? So I removed that if.

Now it works for me.

PS: Even with set_max_bitrate set to false the player patching works in enabling 1080p playback on Linux, if available. Not sure why, just by adding the profile maybe? But I also stumbled over a movie where all of this had no effect, I guess where netflix really does not deliver 1080p playback. Or maybe there are additional playback profiles though ought to be enabled there?

@DavidBuchanan314
Copy link
Owner

PS: Even with set_max_bitrate set to false the player patching works in enabling 1080p playback on Linux, if available. Not sure why, just by adding the profile maybe? But I also stumbled over a movie where all of this had no effect, I guess where netflix really does not deliver 1080p playback. Or maybe there are additional playback profiles though ought to be enabled there?

I don't know for sure, but I believe certain content is configured to require "VMP" (Verified Media Path), which sadly is not available on the Linux platform.

@onli
Copy link

onli commented Mar 24, 2024

That might be it.

Well, I hope the described code changes help with the max bitrate issue at least :)

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

No branches or pull requests

4 participants