[Data Grid] Added closePopover action to EuiDataGridColumnCellActionProps

[sulemanof]

Summary

Allow a custom cell action to close the popover manually.
The original request in Data table vis replacement -> elastic/kibana#70801 (comment)

The desired result:

Checklist

• Props have proper autodocs
• Added documentation
• A changelog entry exists and is marked appropriately

[kibanamachine]

Since this is a community submitted pull request, a Jenkins build has not been kicked off automatically. Can an Elastic organization member please verify the contents of this patch and then kick off a build manually?

[kertal]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_4346/

[sulemanof]

jenkins test this

[kertal]

Code LGTM! Yes, this would fix the problem you've describing, I've encountered the same issue in the Discover PR:

Which makes me wonder if it's intended, that popover cells are still displayed while the parent cell was already removed?

[kertal]

@sulemanof one thing, could you make use of this function in one of the examples of our documentation ? think it's always good to show such things in action, many thx!

[snide]

Echo @kertal. A simple example (probably on the popover's doc page) for the docs would be nice. Although this is a simple prop, a jest test would also be nice. Otherwise feature looks good.

The next release for EUI will be tuesday. That will likely be the last release to get into Kibana.

[sulemanof]

Echo @kertal. A simple example (probably on the popover's doc page) for the docs would be nice. Although this is a simple prop, a jest test would also be nice. Otherwise feature looks good.

The next release for EUI will be tuesday. That will likely be the last release to get into Kibana.

Hey @snide !
I've updated docs for the feature.
Sorry, I wasn't able to add tests here since the component lacks any tests and it requires quite more time to create it therefore.
If we want to get this feature until the FF in kibana, would be great if we could merge it as it is.

[snide]

Talked with @chandlerprall yesterday. He was OK with this one going without tests. Thanks for the docs.

[chandlerprall]

I reverted the cell action example's email column to still alert so it doesn't feel broken and touched up the description. I also changed it to always pass the closePopover method so 1) the consuming app doesn't need to test for it and 2) it is possible to click the non-popover action button when the popover is open

Will merge on passing CI

[chandlerprall]

jenkins test this

[kibanamachine]

Preview documentation changes for this PR: https://eui.elastic.co/pr_4346/