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

Chrome: The provided value middle is not a valid enum value of type AlignSetting #368

Open
ondrejpar opened this issue Oct 28, 2017 · 6 comments

Comments

@ondrejpar
Copy link

When parsing VTT file in Chrome browser, tons of error messages are printed to console:

The provided value middle is not a valid enum value of type AlignSetting

This didn't happen in v0.12.1.

@knopp
Copy link

knopp commented Dec 16, 2017

That seems to be because enum AlignSetting on VTTCue is supposed to have value "center" whereas the parser tries to use "middle"

@ondrejpar
Copy link
Author

Well, yes, but my VTT file doesn't have any align settings at all, so I suppose I get that error because the "middle" value is the default in vtt.js code?

@Addvilz
Copy link

Addvilz commented May 21, 2020

Bump. Any chance we could have this fixed by incorporating a fix similar to videojs/vtt.js#32?

@gkatsev
Copy link
Contributor

gkatsev commented May 21, 2020

Unfortunately, this project isn't really maintained right now. Feel free to use the video.js fork and file issues there.

@Addvilz
Copy link

Addvilz commented May 21, 2020

@gkatsev a little bit offtopic, but is the API in the videojs fork roughly equivalent?

@gkatsev
Copy link
Contributor

gkatsev commented May 21, 2020

The API is pretty much the same. It also works better with bundler, but the on page API right now is equivalent.
I'm currently working on a large refactor of internals along with additions to make the project more spec compatible (like adding current version of regions). Once that's available I'm going to look into rewriting the main API to make it a bit better for working with bundlers.

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

No branches or pull requests

4 participants