-
-
Notifications
You must be signed in to change notification settings - Fork 31.6k
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
[Table] 'onRowSelection' does't get called with 'none' while deselecting all rows by check a single row #5964
Comments
I confirm same behavior on http://www.material-ui.com/#/components/table |
And what is the desired behavior? I can imagine 2 outcomes. Given table with rows [0,1,2,3], click to selectAll, then click on item 2.
And then "selectAll" becomes unchecked. Checking it again will result in selecting all (it both cases) |
I've raised a PR #7453 that fixes this and achieves the desired behavior. |
We have been porting the component on the v1-beta branch. We reimplemented it from the ground-up. While we haven't tested it, I think that the issue is most likely fixed on that branch. Hence, I'm closing it. |
Did this ever get a fix on 0.x branch? Facing this problem on a legacy app that upgrading for the 1.x is not a viable option for now. |
@rdsedmundo No, it wasn't. |
Problem description
Try checking a single row after the table has been allSelected,
the single row won't be checked but 'onRowSelection' will get called with an array, which includes the single row's key.
Link to minimally-working code that reproduces the issue
Versions
The text was updated successfully, but these errors were encountered: