[EuiDataGrid] Removing a row while a popover is open, opens a popover for another row

[myasonik]

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:

[myasonik]

This would improve the UX for Discover's DataGrid reimplementation (elastic/kibana#51531)

[sulemanof]

I suppose this could be closed in favor of #4346
Passing closePopover prop explicitly into consuming application makes the popover closable when needed.

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 rowIndex & columnIndex. The data grid cell doesn't know which data it presents since it's the renderCellValue business.

[chandlerprall]

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 resetState method on the grid's ref, perhaps with a configuration option so e.g. pagination could reset or be left unmodified.

[cee-chen]

Is this issue still valid now that #4346 has effectively solved the problem for Discover? Or can it be closed?

[cee-chen]

This is definitively closed by #5590 since exposing the closeCellPopover API allows consuming applications to manually close a popover when a row is deleted.