-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
IE11 doesn't fire addtrack or removetrack events on native text tracks #2465
Comments
One way to fix it is to force emulated captions for browsers that cannot fire the 'removetrack' in Html5.TEST_VID.textTracks |
Thoughts, @videojs/core-committers |
It looks like IE has an onaddtrack property, but it only works if you use the addTextTrack API...boo. Is there somewhere we can submit a bug for IE? Seems like we'll need the native one if projects like dash.js are interacting with the video element directly, so this seems reasonable. You probably need to change 'removetrack' to 'onremovetrack' |
Oops, yep, I meant |
Looks like IE11 doesn't fir it if we attach the events via |
Yep, looks like Edge is the same issue. |
Can we close this because of #2466? |
Since we filed the bug and disabled native tracks on IE11/Edge, both this and #2438 can be closed. |
I noticed this as part of trying to fix tests in master for #2454. Some of the async tests would hang because these events would never fire.
It would be nice to see if we can fake these events but we probably can't.
The text was updated successfully, but these errors were encountered: