Skip to content

[POC] signVideo w/ YouTube#553

Closed
xerc wants to merge 5 commits intoableplayer:developfrom
xerc:develop
Closed

[POC] signVideo w/ YouTube#553
xerc wants to merge 5 commits intoableplayer:developfrom
xerc:develop

Conversation

@xerc
Copy link
Contributor

@xerc xerc commented Nov 26, 2022

<video
  id="yt0"
  playsinline
  preload="none"
  data-able-player
  data-youtube-nocookie
  data-youtube-id="EcFzdNEkqok"
  data-sign-src="EcFzdNEkqok-sign.mp4"
>

// volume is 0 to 1
newVolume = volume / 10;
this.media.volume = newVolume;

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

keep!

@joedolson
Copy link
Member

Just tested this and it works great; would love to get this merged, @terrill !

joedolson added a commit to joedolson/ableplayer that referenced this pull request Nov 3, 2024
Applies ableplayer#553, and adds missing support for syncing time when scrubbing or clicking in transcript.
@joedolson
Copy link
Member

Merged to develop. Props @xerc

@joedolson joedolson closed this May 26, 2025
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