Skip to content

[EuiFlyout] Introduce EuiFlyoutPluginContext for enhanced integration and deprecate legacy props#9736

Draft
paulinashakirova wants to merge 1 commit into
elastic:mainfrom
paulinashakirova:remove-flyout-history-logic
Draft

[EuiFlyout] Introduce EuiFlyoutPluginContext for enhanced integration and deprecate legacy props#9736
paulinashakirova wants to merge 1 commit into
elastic:mainfrom
paulinashakirova:remove-flyout-history-logic

Conversation

@paulinashakirova

Copy link
Copy Markdown
Contributor
  • Added EuiFlyoutPluginContext to facilitate external package integration with the flyout manager.
  • Introduced menuBarSlot prop to replace the built-in Back button and history popover.
  • Deprecated showBackButton, backButtonProps, and historyItems in favor of the new context.
  • Updated EuiFlyoutMenu and related components to utilize the new context and props.
  • Enhanced tooltip usage in the flyout menu for better accessibility and user experience.

Summary

API Changes

component / parent prop / child change description

Screenshots

Before After
{Before image} {After image}

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 / 🟢 Low / 🟡 Moderate / 🔴 High

Release Readiness

  • Documentation: {link to docs page(s)}
  • 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

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.

… and deprecate legacy props

- Added `EuiFlyoutPluginContext` to facilitate external package integration with the flyout manager.
- Introduced `menuBarSlot` prop to replace the built-in Back button and history popover.
- Deprecated `showBackButton`, `backButtonProps`, and `historyItems` in favor of the new context.
- Updated `EuiFlyoutMenu` and related components to utilize the new context and props.
- Enhanced tooltip usage in the flyout menu for better accessibility and user experience.
@paulinashakirova paulinashakirova self-assigned this Jun 18, 2026
@elasticmachine

Copy link
Copy Markdown
Collaborator

💚 Build Succeeded

cc @paulinashakirova

@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
6 visual difference(s) found - expand to review, then click Approve visual changes to update baselines

euiflyout (4 differences)

StoryBeforeAfterDiff
manual return focus desktop
playground desktop
push flyouts desktop
menu bar example desktop

euipopover (1 difference)

StoryBeforeAfterDiff
playground desktop

euidatagrid (1 difference)

StoryBeforeAfterDiff
additional controls options desktop

@elasticmachine

elasticmachine commented Jun 18, 2026

Copy link
Copy Markdown
Collaborator

💔 Build Failed

Failed CI Steps

History

cc @paulinashakirova

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.

2 participants