-
Notifications
You must be signed in to change notification settings - Fork 842
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
[EuiDataGrid] Removing a row while a popover is open, opens a popover for another row #2625
Comments
This would improve the UX for Discover's DataGrid reimplementation (elastic/kibana#51531) |
I suppose this could be closed in favor of #4346 Note: I don't think it's currently possible to close it automatically when underlying data changes, since a cell is just an item with dimensions |
As Daniil said, EuiDataGrid is unaware of what a change means or represents. We hit this same issue with EuiInMemoryTable as well: when does a change in data represent a different data set. I think the path forward here is to expose a |
Is this issue still valid now that #4346 has effectively solved the problem for Discover? Or can it be closed? |
This is definitively closed by #5590 since exposing the |
If I remove a row from an action in a details popover of a cell, I expect that popover to close and not reopen for the new row at that spot.
Example:
The text was updated successfully, but these errors were encountered: