Skip to content

Conversation

@tushar743-ui
Copy link

@ImprovedTube
Copy link
Member

welcome and thank you so much! @tushar743-ui @MichaelIsGreat123
we have two features for this on top of the player section.
( As one they are mutually exclusive, when enabling both, the earlier toggle is disabled again. )

Certainly many people use this, but some reviews said it was unreliable, so it will be great to consider at least Chrome and Firefox and any secondary code to fall back to.

I will agree in enabling this by default for every single installation, because it is nice,
if we think hard about possible adverse effects. Of course everything by default will cause some confusion,
but here the purpose seems to outweigh by far.

https://github.com/code-charity/youtube/blob/72dd0ac094aac26a6be2d42beb54ca5180f88f99/js%26css/web-accessible/www.youtube.com/player.js#L54-L69

https://github.com/code-charity/youtube/blob/72dd0ac094aac26a6be2d42beb54ca5180f88f99/js%26css/extension/www.youtube.com/general/general.js#L169-L180

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.

2 participants