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

docs: add note for Safari captions error #174

Merged
merged 3 commits into from
Jul 30, 2018

Conversation

forbesjo
Copy link
Contributor

Update documentation to handle a Safari decode error scenario

@@ -37,3 +38,10 @@ you need to properly configure CORS on your server: https://github.com/videojs/h
Some browsers support native playback of certain streaming formats. By default, we defer to the native players. However, this means that features specific to videojs-http-streaming will not be available.

Solution: use videojs-http-streaming based playback on those devices: https://github.com/videojs/http-streaming#overridenative

## media_err_decode error on Desktop Safari
Copy link
Contributor

Choose a reason for hiding this comment

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

Probably worth using all caps since the error comes back in all caps


## media_err_decode error on Desktop Safari

This error may occur if the source manifest has `CLOSED-CAPTIONS=NONE` and an external text track is loaded into the player. Safari does not allow the inclusion any captions if the manifest indicates that captions will not be provided.
Copy link
Contributor

Choose a reason for hiding this comment

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

We may also want to mention that MEDIA_ERR_DECODE can happen for a number of reasons, as it is particularly common for misconfigured content, the captions being just one of them.

@@ -9,6 +9,7 @@ For issues around data embedded into media segments (e.g., 608 captions), see th
- ["No compatible source was found" on IE11 Win 7](#no-compatible-source-was-found-on-ie11-win-7)
- [CORS: No Access-Control-Allow-Origin header](#cors-no-access-control-allow-origin-header)
- [Desktop Safari/iOS Safari/Android Chrome/Edge exhibit different behavior from other browsers](#desktop-safariios-safariandroid-chromeedge-exhibit-different-behavior-from-other-browsers)
- [media_err_decode error on Desktop Safari](#media_err_decode-error-on-desktop-safari)
Copy link
Contributor

Choose a reason for hiding this comment

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

One more spot for all caps

@forbesjo forbesjo merged commit 7b03530 into videojs:master Jul 30, 2018
@forbesjo forbesjo deleted the captions-error branch July 30, 2018 21:41
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