Skip to content

Conversation

@tsullivan
Copy link
Member

Summary

Setting ownFocus=false in Flyout System storybooks where the user needs access to the underlying page to open additional flyouts into the Flyout System.

Why are we making this change?

This is a change that is needed for testing.

Screenshots #

Impact to users

Only developers testing the Flyout System will be impacted.

QA

Remove or strikethrough items that do not apply to your PR.

General checklist

  • Browser QA
    • Checked in both light and dark modes
    • Checked in both MacOS and Windows high contrast modes
    • Checked in mobile
    • Checked in Chrome, Safari, Edge, and Firefox
    • Checked for accessibility including keyboard-only and screenreader modes
  • Docs site QA
  • Code quality checklist
  • Release checklist
    • A changelog entry exists and is marked appropriately.
    • If applicable, added the breaking change issue label (and filled out the breaking change checklist)
  • Designer checklist
    • If applicable, file an issue to update EUI's Figma library with any corresponding UI changes. (This is an internal repo, if you are external to Elastic, ask a maintainer to submit this request)

@tsullivan tsullivan requested a review from a team as a code owner October 16, 2025 18:15
@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@elasticmachine
Copy link
Collaborator

💚 Build Succeeded

@tkajtoch tkajtoch self-requested a review October 17, 2025 14:38
Copy link
Member

@tkajtoch tkajtoch left a comment

Choose a reason for hiding this comment

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

LGTM

@tsullivan tsullivan merged commit 848b26a into elastic:feat/flyout-system Oct 17, 2025
5 checks passed
@tsullivan tsullivan deleted the flyout-system/fix-storybook-ownfocus branch October 17, 2025 21:12
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.

3 participants