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

add Media Source Extension API feature dectection #2233

Merged
merged 2 commits into from
Jul 3, 2018

Conversation

haraldreingruber
Copy link
Contributor

@haraldreingruber haraldreingruber commented Aug 7, 2017

"name": "MDN documentation",
"href": "https://developer.mozilla.org/en-US/docs/Web/API/Media_Source_Extensions_API"
}],
"builderAliases": ["media_source_extension_api"]
Copy link
Member

Choose a reason for hiding this comment

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

could you add a caniuse tag?

Detects support the Media Source Extensions API, which allows JavaScript to send byte streams to media codecs within web browsers that support HTML5 video.
*/
define(['Modernizr'], function(Modernizr) {
// https://developer.mozilla.org/en-US/docs/Web/API/Media_Source_Extensions_API
Copy link
Member

Choose a reason for hiding this comment

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

Not needed here again.

@rejas rejas added this to the Modernizr v3.8 milestone Jun 29, 2018
@rejas
Copy link
Member

rejas commented Jun 29, 2018

@patrickkettner would love to hear you opinion on this too

@rejas rejas requested a review from patrickkettner June 29, 2018 12:40
@patrickkettner
Copy link
Member

@rejas you covered the only issues I see. Once fixed, lgtm

@haraldreingruber
Copy link
Contributor Author

Thanks for your feedback @rejas.
Could you check if my changes meet your suggestions?

@rejas rejas merged commit 4e56bfd into Modernizr:master Jul 3, 2018
@rejas
Copy link
Member

rejas commented Jul 3, 2018

Looked good. Thx for your work!

@rejas rejas modified the milestones: Modernizr v3.8, Modernizr v3.7 Jul 3, 2018
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.

3 participants