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

Update player.js more subtitle safety #2400

Merged
merged 4 commits into from
Jun 20, 2024
Merged

Update player.js more subtitle safety #2400

merged 4 commits into from
Jun 20, 2024

Conversation

raszpl
Copy link
Contributor

@raszpl raszpl commented Jun 19, 2024

Continuing #2385 . youtube is leaving shadow inactive player on navigating to subscriptions/home, was tripping up subtitle routines

youtube is leaving shadow inactive player on navigating to subscriptions/home, was tripping up subtitle routines
@raszpl raszpl marked this pull request as draft June 19, 2024 23:36
showing actual state of subtitles. YT doesnt have a function returning status of captions currently loaded, checking button color is the only way I could find :(
@raszpl raszpl marked this pull request as ready for review June 20, 2024 00:16
@raszpl raszpl marked this pull request as draft June 20, 2024 09:51
@raszpl raszpl marked this pull request as ready for review June 20, 2024 10:40
@ImprovedTube
Copy link
Member

shadow inactive player

Glad to read that though! 😆 (Since overcome surprises /frustration, are special preparation and can conclude in special motivation, to keep making this project ever nicer / faster than anyone else could)
(Which also motivates/increases mine)

@ImprovedTube ImprovedTube merged commit 0601bf6 into code-charity:master Jun 20, 2024
1 check passed
@raszpl raszpl deleted the patch-8 branch June 20, 2024 18:26
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