-
Notifications
You must be signed in to change notification settings - Fork 17
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
Fix some issues in the datatable #888
Conversation
Checkbox div clickable
7770d00
to
f178bcd
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #888 +/- ##
=======================================
Coverage 68.22% 68.22%
=======================================
Files 62 62
Lines 3909 3909
=======================================
Hits 2667 2667
Misses 1242 1242 |
datalab Run #2428
Run Properties:
|
Project |
datalab
|
Branch Review |
bc/datatable-fix
|
Run status |
Passed #2428
|
Run duration | 05m 59s |
Commit |
00a76d508e ℹ️: Merge ef47ab418030d0c52e530fd4f3aff6f92ec58ff1 into f7c555073f10bb591c419ee81395...
|
Committer | Benjamin Charmes |
View all properties for this run ↗︎ |
Test results | |
---|---|
Failures |
0
|
Flaky |
0
|
Pending |
0
|
Skipped |
0
|
Passing |
393
|
View all changes introduced in this branch ↗︎ |
Fix selectall checkbox in DataTable
65d4eb3
to
d0f8042
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This looks great @BenjaminCharmes! Did something get fixed with the wrapping of long names too? It seems to work nicely for me now
I just had a couple of really minor suggestions below; I think I will commit these myself and then merge -- perhaps you could tidy up the ID validation in JS since we duplicate it in a few places now?
Fix all-select checkbox Default row: 20 Apply suggestions from code review
4801d49
to
ef47ab4
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍
Addresses #889