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

When CTRL is pressed always ask #59

Closed
wants to merge 2 commits into from
Closed

Conversation

noximo
Copy link

@noximo noximo commented Mar 6, 2023

Added functionality that overrides preselected mode to "ask" when CTRL is pressed while entering fullscreen.

The popup shows info when it becomes applicable.

Readme was updated to reference this new functionality.

Ngl, javascript isn't my language of choice, async/awaits are completely unknown to me so I wasn't able to utilize them (I don't even know if I should have). I understand if the code isn't on par with the rest of the codebase, but this is the best I can do. That said, all works fine on my end so hopefully, I did not introduce any problems.

Added a functionality that overrides preselected mode to "ask" when CTRL is pressed while entering fullscreen.
@BradB111
Copy link

BradB111 commented May 4, 2024

I was looking for exactly this. Thank you for taking the time to develop this and hope this is merged in future releases.

@dralletje dralletje closed this Oct 29, 2024
@dralletje
Copy link
Owner

This is in the latest version released on Chrome!

@BradB111
Copy link

BradB111 commented Oct 29, 2024

This is in the latest version released on Chrome!

Unfortunately this doesn't seem to be working on Youtube at least in my testing now on Version 32.

Selecting fullscreen as a default works however shift+click or ctrl+click on the fullscreen button on the player does not trigger the popup asking which fullscreen method to use.

@dralletje
Copy link
Owner

OOo very stupid! I forgot about fullscreen 😂 (I disable the extension more when you have fullscreen selected, in hopes that I break less websites... but that is going out I guess!)

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