Fixed bug in EuiBasicTable to handle dynamic icon value properly in collapsed actions

[crux153]

Summary

Fix for #3144

While DefaultItemAction accepts a function to dynamically change icon value as like below,

type IconFunction<T> = (item: T) => EuiIconType;
export interface DefaultItemIconButtonAction<T>
  extends DefaultItemActionBase<T> {
  type: 'icon';
  icon: EuiIconType | IconFunction<T>;
  color?: EuiButtonIconColor | EuiButtonIconColorFunction<T>;
}

It crashes when shown in CollapsedItemActions component, since the logic to handle functional icon prop is present only in DefaultItemAction and missing in CollapsedItemActions.

{
    name: 'Clone',
    description: 'Clone this user',
    icon: item => (item.online ? 'eye' : 'eyeClosed'),
    onClick: this.cloneUser,
    'data-test-subj': 'action-clone',
}

If I pass an action with dynamic icon value like above, the table crashes when clicking the expand button in action column.

This PR adds the missing handler to CollapsedItemActions so that it works in the same way as DefaultItemAction does.

Checklist

• Check against all themes for compatibility in both light and dark modes
• Checked in mobile
• Checked in IE11 and Firefox
• Props have proper autodocs
• Added documentation examples
• Added or updated jest tests
• Checked for breaking changes and labeled appropriately
• Checked for accessibility including keyboard-only and screenreader modes
• 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?

[cchaos]

@crux153 Can you please give more explanation in the summary?

[crux153]

@cchaos I updated the summary. Thanks!

[chandlerprall]

Nice! Thanks for digging into this and finding the root cause. I've asked for two small changes, otherwise this looks great.

[crux153]

@chandlerprall I just copied the implementation detail from existing DefaultItemAction component. I love your setup as it is much cleaner than before, so I applied the same changes to both components. Thank you :)

[chandlerprall]

jenkins test this

[kibanamachine]

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

[chandlerprall]

Thanks for tracking down the root cause, and the extra cleanup on this one!