[EuiDataGrid] Control isExpandable at the cell level

[pickypg]

I am building a calendar using the DataGrid and I have run into the need to allow a small popover form, but only if the calendar day has stuff in it. For example, imagine that a manager wants to override a holiday to mark an engineer as working, it obviously is an unnecessary step if there is no holiday on that day.

I'd like an enhancement where, before it attempts to render the cell with isDetails set to true, (or the popover version), it would instead check if it even should. The easiest approach seems to be making isExpandable a callback | boolean | undefined, which probably helps it to avoid invoking a function call for all cells anyway. For the best impact, this function would need be passed the rowIndex and columnId just like the render cell function.

[pickypg]

/cc @chandlerprall

[pickypg]

I'd like an enhancement where, before it attempts to render the cell with isDetails set to true, (or the popover version), it would instead check if it even should.

Before it even shows the expand button that is.

[github-actions]

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

[pickypg]

This still seems useful.

[github-actions]

👋 Hey there. This issue hasn't had any activity for 180 days. We'll automatically close it if that trend continues for another week. If you feel this issue is still valid and needs attention please let us know with a comment.

[github-actions]

❌ We're automatically closing this issue due to lack of activity. Please comment if you feel this was done in error.

[cee-chen]

@pickypg 👋 Heyyy!! I've recently joined the EUI team and am helping out with EuiDataGrid - I hadn't seen this issue until it closed just now. Just curious, is it still useful to the calendar you're building? No promises, but I'd be interested in picking this up at some point if so :)

[pickypg]

Hey @constancecchen, it would! However, we currently use the EuiTable in place of the datagrid. We would be open to checking it out again, but I think this feature would be useful regardless.

[cee-chen]

Cool cool, great to know! As a heads up I'm currently prioritizing issues/features that teams need or are requesting in order to roll out features (currently some from Discover and Security), so it might be a while for this - but if you decide to switch over to EuiDataGrid or that this would be a blocker for doing so, definitely let me know and feel free to re-open this issue whenever!

[robert-seymour-numerated]

@constancecchen Hello! This feature just came up in our teams standup today. Would be really useful to to control isExpandable on a cell level. Consider a simple id table: id=1 should not be expandable. but id=12312312313 would be nice.

[cee-chen]

Nice! I'll re-open as this is a recent re-request, but to level-set expectations, we've got a whole lot on the datagrid backlog so it might be a while yet before this gets looked at!