-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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): Add closest ponyfill #3559
Conversation
fe5ff3f
to
1e584eb
Compare
Codecov Report
@@ Coverage Diff @@
## master #3559 +/- ##
========================================
Coverage ? 98.4%
========================================
Files ? 120
Lines ? 5067
Branches ? 629
========================================
Hits ? 4986
Misses ? 81
Partials ? 0
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3559 +/- ##
=========================================
Coverage ? 98.42%
=========================================
Files ? 120
Lines ? 5131
Branches ? 636
=========================================
Hits ? 5050
Misses ? 81
Partials ? 0
Continue to review full report at Codecov.
|
2 similar comments
Codecov Report
@@ Coverage Diff @@
## master #3559 +/- ##
=========================================
Coverage ? 98.42%
=========================================
Files ? 120
Lines ? 5131
Branches ? 636
=========================================
Hits ? 5050
Misses ? 81
Partials ? 0
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #3559 +/- ##
=========================================
Coverage ? 98.42%
=========================================
Files ? 120
Lines ? 5131
Branches ? 636
=========================================
Hits ? 5050
Misses ? 81
Partials ? 0
Continue to review full report at Codecov.
|
Fixes #3536.
I used td for unit tests since otherwise running them locally shows a huge gap (since Chrome supports native
closest
so the entire other branch never gets hit).