Skip to content

[EuiToolTip] Update documentation#9640

Merged
weronikaolejniczak merged 2 commits intoelastic:mainfrom
weronikaolejniczak:feat/tooltip-docs
May 7, 2026
Merged

[EuiToolTip] Update documentation#9640
weronikaolejniczak merged 2 commits intoelastic:mainfrom
weronikaolejniczak:feat/tooltip-docs

Conversation

@weronikaolejniczak
Copy link
Copy Markdown
Contributor

@weronikaolejniczak weronikaolejniczak commented May 6, 2026

Summary

  • What: Updated tooltip documentation with new guidance:
    • native browser tooltips warning,
    • accessibility requirements success criterion links,
    • disabled elements pattern with interactive demo.
  • Why: Closes [Tooltips] Update EUI documentation #9568

API Changes

N/A

Screenshots

N/A

Impact Assessment

Note: Most PRs should be tested in Kibana to help gauge their Impact before merging.

  • 🔴 Breaking changes — What will break? How many usages in Kibana/Cloud UI are impacted?
  • 💅 Visual changes — May impact style overrides; could require visual testing. Explain and estimate impact.
  • 🧪 Test impact — May break functional or snapshot tests (e.g., HTML structure, class names, default values).
  • 🔧 Hard to integrate — If changes require substantial updates to Kibana, please stage the changes and link them here.

Impact level: 🟢 None

Release Readiness

  • Documentation: EuiToolTip
  • Figma: {link to Figma or issue}
  • Migration guide: {steps or link, for breaking/visual changes or deprecations}
  • Adoption plan (new features): {link to issue/doc or outline who will integrate this and where}

QA instructions for reviewer

  • Verify the documentation is comprehensive
  • Verify there are no typos
  • Check if all examples and sandboxes work

Checklist before marking Ready for Review

Reviewer checklist

  • Approved Impact Assessment — Acceptable to merge given the consumer impact.
  • Approved Release Readiness — Docs, Figma, and migration info are sufficient to ship.

@weronikaolejniczak weronikaolejniczak self-assigned this May 6, 2026
@weronikaolejniczak weronikaolejniczak marked this pull request as ready for review May 6, 2026 16:33
@weronikaolejniczak weronikaolejniczak requested a review from a team as a code owner May 6, 2026 16:33
Copilot AI review requested due to automatic review settings May 6, 2026 16:33
@acstll acstll self-requested a review May 7, 2026 17:12
Copy link
Copy Markdown
Contributor

@acstll acstll left a comment

Choose a reason for hiding this comment

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

🟢 Read new content and tested new demos. Looks good to merge, only left a nit. 🚢

Comment thread packages/website/docs/components/display/tooltip.mdx Outdated
Co-authored-by: Arturo Castillo Delgado <arturo@arturu.com>
@weronikaolejniczak weronikaolejniczak enabled auto-merge (squash) May 7, 2026 17:21
@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

cc @weronikaolejniczak

@elasticmachine
Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

History

cc @weronikaolejniczak

@weronikaolejniczak weronikaolejniczak merged commit c39856d into elastic:main May 7, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Tooltips] Update EUI documentation

3 participants