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

feat(dom): Create mdc-dom package with Element.matches() ponyfill #3515

Merged
merged 12 commits into from
Sep 5, 2018

Conversation

acdvorak
Copy link
Contributor

@acdvorak acdvorak commented Sep 5, 2018

Will be used by #3413 and eventually other packages (e.g., for Element.closest() ponyfill)

Refs #1104

@codecov-io
Copy link

codecov-io commented Sep 5, 2018

Codecov Report

Merging #3515 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #3515      +/-   ##
==========================================
+ Coverage    98.4%   98.41%   +<.01%     
==========================================
  Files         119      120       +1     
  Lines        5028     5033       +5     
  Branches      618      621       +3     
==========================================
+ Hits         4948     4953       +5     
  Misses         80       80
Impacted Files Coverage Δ
packages/mdc-dom/ponyfill.js 100% <100%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f3215e6...dd5ebe5. Read the comment docs.

@mdc-web-bot
Copy link
Collaborator

All 389 screenshot tests passed for commit 91a61c1 vs. master! 💯🎉

@mdc-web-bot
Copy link
Collaborator

All 389 screenshot tests passed for commit b4c9249 vs. master! 💯🎉

@mdc-web-bot
Copy link
Collaborator

All 389 screenshot tests passed for commit 06dd7d6 vs. master! 💯🎉

@mdc-web-bot
Copy link
Collaborator

All 389 screenshot tests passed for commit dd5ebe5 vs. master! 💯🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants