Skip to content

[EuiCopy] Add tooltipProps prop #8382

@weronikaolejniczak

Description

@weronikaolejniczak

Description

The purpose of this task is to add a tooltipProps prop to the EuiCopy component instead of passing tooltip props flat to the component type. This helps to to distinguish tooltip props from the copy functionality props.

It's consistent with other components that consume props to pass to children (e.g. for EuiSuperUpdateButton (code) or EuiButtonGroupButton (code))

Important

Updating the component API will be a breaking change and requires updating implementation usages.

Acceptance criteria

  • added prop tooltipProps to EuiCopy and updated it's prop type definition
  • updated usages/examples across EUI docs
  • tested/verified the API change for usages in Kibana (pre-prepared changes for release upgrade)
  • Since we need to update tests for this anyway, it is probably worth migrating to do as RTL at the same time, assuming it's a small test file.

Rollout consideration

Note on impact / scope of rollout: If we make it a breaking change, there is usage in Kibana we would need to update, but it should be limited and we can handle that ourselves as part of the upgrade.

Metadata

Metadata

Labels

breaking changePRs with breaking changes. (Don't delete - used for automation)low hanging fruitAn issue, often a bug, that is lower effort and clearly ought to be fixed
No fields configured for Enhancement.

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions