-
Notifications
You must be signed in to change notification settings - Fork 7.5k
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
Track metadata issues with video.js#4.12.12 #2436
Comments
@gkatsev is this related to the one already being fixed or is it new? At least sounds related to #2412. @chemoish if you want to try moving to 5.0, we're not mangling anymore so shouldn't be an issue. http://jsbin.com/sofiwu/edit?html,console,output |
Yeah, it's a silly mistake. We could probably fix it as part of #2412 (yay gcc...) |
This should be fixed in stable now. a02f0be |
@gkatsev I am receiving an undefined track after this update.
Can you verify? https://github.com/videojs/video.js/blob/stable/src/js/tracks/text-track-list.js#L78 Captions seem to be turned on by default (Instead of the other way around). However, it looks like you can turn them off via the control bar. |
Moving this to a new issue. |
This works on
Video.dev.js#4.12.12
.Video.js#4.12.12
throws the following:Maybe:
The text was updated successfully, but these errors were encountered: