-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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: Fix selection loss when applying filters and using Items property #8538
Table: Fix selection loss when applying filters and using Items property #8538
Conversation
Rename multiselect test for ServerData
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## dev #8538 +/- ##
==========================================
+ Coverage 89.50% 89.54% +0.04%
==========================================
Files 411 411
Lines 11832 11832
Branches 2349 2349
==========================================
+ Hits 10590 10595 +5
+ Misses 718 715 -3
+ Partials 524 522 -2 ☔ View full report in Codecov by Sentry. |
@ralvarezing can you rebase / merge against latest changes? The datepicker tests were fixed. |
Perfect! done |
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.
LGTM
Thanks @ralvarezing |
Description
The multiselect were being cleared when a filter or page changes using Items.
This was caused because when evaluating if the disposed table item is in the table, was evaluating against the FilteredItems instead of Items.
Because of this, when a filter was applied, all items that didn't match the filter, were consider NOT to be in the table, and being removed from the SelectedItems.
A similar bug was fixed in #7914, but when the data source is ServerData.
fixes #7535
How Has This Been Tested?
Added test TableMultiSelectItemsTest1_PageChange
Added test TableMultiSelectItemsTest1_FilterChange
Types of changes
Checklist:
dev
).