Skip to content

feat(obs-exploration-team): replace title props and wrap EuiButtonIcon with EuiToolTip#271494

Open
weronikaolejniczak wants to merge 3 commits into
elastic:mainfrom
weronikaolejniczak:feat/replace-title-obs-exploration-team
Open

feat(obs-exploration-team): replace title props and wrap EuiButtonIcon with EuiToolTip#271494
weronikaolejniczak wants to merge 3 commits into
elastic:mainfrom
weronikaolejniczak:feat/replace-title-obs-exploration-team

Conversation

@weronikaolejniczak
Copy link
Copy Markdown
Contributor

@weronikaolejniczak weronikaolejniczak commented May 27, 2026

Closes #270154

Relates to elastic/eui#9566

Important

These changes should be carefully tested visually by each code owner. Wrapping with EuiToolTip instead of passing title leads to another DOM node and can potentially break the layout. In such cases, I would appreciate committing appropriate fixes to this PR directly, I cannot possibly setup and run all Kibana functionalities to fix every regression.

This PR:

  • wraps ALL EuiButtonIcon with EuiToolTip, the content is the same as aria-label, any title passed to EuiButtonIcon is removed,
  • changes several title cases (not truncation related) to use EuiToolTip instead (if applicable).

@weronikaolejniczak weronikaolejniczak added backport:skip This PR does not require backporting EUI release_note:skip Skip the PR/issue when compiling release notes labels May 27, 2026
@weronikaolejniczak weronikaolejniczak marked this pull request as ready for review May 28, 2026 09:12
@weronikaolejniczak weronikaolejniczak requested a review from a team as a code owner May 28, 2026 09:12
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/eui-team (EUI)

@alexwizp alexwizp added backport:version Backport to applied version labels v9.5.0 v9.4.2 and removed backport:skip This PR does not require backporting labels May 28, 2026
@kibanamachine
Copy link
Copy Markdown
Contributor

💛 Build succeeded, but was flaky

Failed CI Steps

Metrics [docs]

Async chunks

Total size of all lazy-loaded chunks that will be downloaded as the user navigates the app

id before after diff
discover 1.9MB 1.9MB +57.0B
infra 1.5MB 1.5MB +264.0B
logsShared 103.0KB 103.1KB +57.0B
total +378.0B

History

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:version Backport to applied version labels EUI release_note:skip Skip the PR/issue when compiling release notes v9.4.2 v9.5.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Metrics][Discover] Add Kibana tooltips to metrics in Discover capabilities

3 participants