Skip to content

feat(kibana-visualizations): replace title props and wrap EuiButtonIcon with EuiToolTip#271490

Open
weronikaolejniczak wants to merge 12 commits into
elastic:mainfrom
weronikaolejniczak:feat/replace-title-kibana-visualizations
Open

feat(kibana-visualizations): replace title props and wrap EuiButtonIcon with EuiToolTip#271490
weronikaolejniczak wants to merge 12 commits into
elastic:mainfrom
weronikaolejniczak:feat/replace-title-kibana-visualizations

Conversation

@weronikaolejniczak
Copy link
Copy Markdown
Contributor

@weronikaolejniczak weronikaolejniczak commented May 27, 2026

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
@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
@alexwizp
Copy link
Copy Markdown
Contributor

/ci

@alexwizp
Copy link
Copy Markdown
Contributor

/ci

@alexwizp
Copy link
Copy Markdown
Contributor

/ci

@weronikaolejniczak weronikaolejniczak force-pushed the feat/replace-title-kibana-visualizations branch from a74e2da to d252ac6 Compare May 29, 2026 10:09
@weronikaolejniczak weronikaolejniczak force-pushed the feat/replace-title-kibana-visualizations branch from 16e1d7c to 6c20d13 Compare May 29, 2026 11:11
@kibanamachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

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
eventAnnotationListing 208.9KB 209.2KB +327.0B
lens 2.1MB 2.1MB +1.8KB
visDefaultEditor 81.9KB 82.5KB +704.0B
visTypeTimeseries 441.0KB 441.4KB +463.0B
visTypeVega 2.0MB 2.0MB +329.0B
total +3.6KB

Page load bundle

Size of the bundles that are downloaded on every page load. Target size is below 100kb

id before after diff
visDefaultEditor 32.3KB 32.3KB +57.0B
visTypeVega 35.8KB 36.0KB +206.0B
total +263.0B
Unknown metric groups

ESLint disabled line counts

id before after diff
charts 0 1 +1

Total ESLint disabled count

id before after diff
charts 0 1 +1

History

@weronikaolejniczak weronikaolejniczak marked this pull request as ready for review May 29, 2026 12:09
@weronikaolejniczak weronikaolejniczak requested a review from a team as a code owner May 29, 2026 12:09
@infra-vault-gh-plugin-prod
Copy link
Copy Markdown

Pinging @elastic/eui-team (EUI)

@andrimal andrimal self-requested a review May 29, 2026 15:34
Copy link
Copy Markdown
Contributor

@andrimal andrimal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested locally and everything seems working as expected ✨

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

examples:
Image

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image Image Image Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ checked:

Image

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.

4 participants