Closed softage0 closed 7 years ago
@@ Coverage Diff @@
## master #4 +/- ##
=========================================
+ Coverage 88.88% 100% +11.11%
=========================================
Files 16 16
Lines 234 232 -2
Branches 90 90
=========================================
+ Hits 208 232 +24
+ Misses 26 0 -26
Impacted Files | Coverage Δ | |
---|---|---|
src/filter.js | 100% <100%> (ø) |
:white_check_mark: |
src/toggle-children.js | 100% <100%> (+96.29%) |
:white_check_mark: |
src/get-parents.js | 100% <100%> (ø) |
:white_check_mark: |
src/search.js | 100% <100%> (ø) |
:white_check_mark: |
src/get-level.js | 100% <100%> (ø) |
:white_check_mark: |
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 2b08d50...290e04c. Read the comment docs.
oh only toggle-children.js
test is missed :(
It looks I need to add test case first lol
Yeah, good after a test case.
Test cases has been added. This PR also fixes Issue #3.
Published as 3.0. Thanks. 👍
Cool!
It's little late to say that, but thanks for your work reactabular
that includes all features what I need from React-based table :)
No probs. Note that I changed the index treatment of toggling to match the old behavior so you may have to write a callback. 👍
Since
toggleChildren
doesn't passparentField
tohasChildren
method, className is not decided correctly.It fixed by passing
parentField
tohasChildren
method.