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

document navigation, ad and player-updated hooks #1312

Closed
wants to merge 7 commits into from

Conversation

mchangrh
Copy link
Contributor

@mchangrh mchangrh commented May 9, 2022

  • I agree to license my contribution under LGPL-3.0 or my contribution is from another project with a license compatible with LGPL-3.0

To test this pull request, follow the instructions in the wiki.


blocked by #1258 because of videoInfo base
closes #311
closes #1278

onInvidous and onMobileYoutube were kept since they didn't offer any shortcuts and the longer lookup decreased readability

mchangrh and others added 7 commits March 31, 2022 21:58
- purge getVideoInfo as data structure is outdated + currently impossible ajayyy#1257
- remove additional regex check on getChannelID
- add pageType to shortcut videoID lookup when looking at channel/ browse/ search/ shorts/ watch
- removed unnecessary document clones
- reset videoInfo and channelInfo to empty objects instead
- use onInvidious to only signify if a page is on invidious
- hook into playerClient for reliable isLive/ isPremiere info
@mchangrh mchangrh changed the title add document hooks document navigation, ad and player-updated hooks May 9, 2022
mchangrh added a commit to mchangrh/SponsorBlock that referenced this pull request Jul 29, 2022
@mchangrh mchangrh closed this Jul 29, 2022
@mchangrh mchangrh deleted the contentScript branch August 3, 2022 16:58
ajayyy pushed a commit to mchangrh/SponsorBlock that referenced this pull request Sep 2, 2022
@mchangrh mchangrh mentioned this pull request Dec 4, 2022
4 tasks
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.

YouTube event hooks Listen to YouTube polymer events
2 participants