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

Mediastream feature detect #2669

Merged
merged 8 commits into from
Oct 14, 2021
Merged

Mediastream feature detect #2669

merged 8 commits into from
Oct 14, 2021

Conversation

onkardahale
Copy link
Contributor

@onkardahale onkardahale commented Oct 6, 2021

With reference to issue #2577

@Markel @rejas @patrickkettner
Please let me know if there are any fixes to be done

Copy link
Contributor

@Markel Markel left a comment

Choose a reason for hiding this comment

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

Oh! Nice to see this issue being worked on again :)

Seems okay to me (note: I've only checked visually, haven't tested it on a browser)

Anyway, it would be nice for @patrickkettner to check it too, would like his opinion about merging this 🙃

Copy link
Member

@rejas rejas left a comment

Choose a reason for hiding this comment

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

looks good, just some small changes needed

lib/config-all.json Outdated Show resolved Hide resolved
@rejas rejas merged commit 049b6ca into Modernizr:master Oct 14, 2021
@rejas
Copy link
Member

rejas commented Oct 14, 2021

Thx a lot!!!

rejas pushed a commit that referenced this pull request Oct 14, 2021
* Create mediastream.js

* added mediarecorder as test

* Update config-all.json

* Added metadata to mediastream.js

* Update mediastream.js

* Removed extra line, added a new tag 'media'

* moved mediastream.js to /webrtc dir

* Update config-all.json
rejas pushed a commit that referenced this pull request Oct 14, 2021
* Create mediastream.js

* added mediarecorder as test

* Update config-all.json

* Added metadata to mediastream.js

* Update mediastream.js

* Removed extra line, added a new tag 'media'

* moved mediastream.js to /webrtc dir

* Update config-all.json
rejas pushed a commit that referenced this pull request Jan 30, 2022
* Create mediastream.js

* added mediarecorder as test

* Update config-all.json

* Added metadata to mediastream.js

* Update mediastream.js

* Removed extra line, added a new tag 'media'

* moved mediastream.js to /webrtc dir

* Update config-all.json
rejas added a commit that referenced this pull request Jan 30, 2022
* Add avif detection (#2539)

* Add avif detection

* Add tests

Both the integration test and the caniuse test

* Add polyfill

* Add support for ResizeObserver (#2582)

* Add Clipboard API (#2573)

* Add Clipboard API testing

* Implement sub-modules (but badly)

* Solve ESLint errors 🙄

* Improve the sub-properties detection

* Move feature detect location

* Add detection for support of scrollToOptions mode of window.scrollTo() (#2575)

* Add detect/tests for prefetch (#2610)

* Add detect/tests for prefetch

* Move to link directory

* Add in check for ie11 edge case

* Add feature detection for IndexedDB 2.0 (#2607)

* Add a|area[ping] detection (#2671)

* Add a|area[ping] detect

* add aping|areaping to integration tests

* Add author tag

* Add Mediastream feature detect (#2669)

* Create mediastream.js

* added mediarecorder as test

* Update config-all.json

* Added metadata to mediastream.js

* Update mediastream.js

* Removed extra line, added a new tag 'media'

* moved mediastream.js to /webrtc dir

* Update config-all.json

* Add test for :focus-visible and fix wrong references in :focus-within (#2665)

* Create focusvisible.js

* Fix wrong reference to CSS focus-within feature

* Fix wrong reference for CSS focus-within feature

* Add css-focus-visible to caniuse integration tests

* Include focusvisible in config-all.json

* Add authors tag to focusvisible.js

Co-authored-by: Markel F <github@markel.dev>
Co-authored-by: Christian Andersson <chris.chris.90@hotmail.com>
Co-authored-by: Chris Smith <chris13524@users.noreply.github.com>
Co-authored-by: Jordan Tepper <jstepper91@gmail.com>
Co-authored-by: Tan Zhen Yong <Xenonym@users.noreply.github.com>
Co-authored-by: Hélio Correia <dev@heliocorreia.org>
Co-authored-by: Onkar Dahale <dahaleonkar@gmail.com>
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