Thanks to visit codestin.com
Credit goes to github.com

Skip to content

fix(EuiFlyoutBody): add fullHeight prop for height-dependent content#9414

Open
ragini-pandey wants to merge 4 commits intoelastic:mainfrom
ragini-pandey:fix/8691-eui-flyout-body-full-height
Open

fix(EuiFlyoutBody): add fullHeight prop for height-dependent content#9414
ragini-pandey wants to merge 4 commits intoelastic:mainfrom
ragini-pandey:fix/8691-eui-flyout-body-full-height

Conversation

@ragini-pandey
Copy link
Contributor

@ragini-pandey ragini-pandey commented Feb 26, 2026

Summary

Adds a new fullHeight prop to EuiFlyoutBody that applies display: flex, flex-direction: column, and height: 100% to the inner .euiFlyoutBody__overflowContent container, fixing height-dependent content (e.g. EuiSelectable height="full") rendering with zero height inside a flyout body.

Why are we making this change?

Fixes #8691.

EuiSelectable with height="full" (virtualized list) renders with a height of 0 when placed inside EuiFlyoutBody. The .euiFlyoutBody__overflowContent wrapper lacks display: flex and full height, preventing flex-based height resolution from propagating to children.

The existing workaround required consumers to manually apply custom CSS:

<EuiFlyoutBody css={css`
  .euiFlyoutBody__overflowContent { display: flex; height: 100%; }
`}>

This PR exposes a first-class fullHeight prop instead:

<EuiFlyoutBody fullHeight>
  <EuiSelectable height="full" options={options}>
    {list => list}
  </EuiSelectable>
</EuiFlyoutBody>

Screenshots

Screenshot 2026-02-26 at 11 57 47 AM

N/A — this is a layout/CSS fix. The visual result is that EuiSelectable (and other height-dependent virtualized components) now renders correctly at full height inside EuiFlyoutBody when fullHeight is set.

Impact to users

Low impact, additive change only. The new fullHeight prop defaults to false, so there is no change in behaviour for existing consumers. Users who need virtualized or height-dependent content inside EuiFlyoutBody can now opt in with fullHeight.

QA

Can be tested here -> http://localhost:6006/?path=/story/layout-euiflyout-euiflyoutbody--full-height

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
    • Added documentation
    • Props have proper autodocs (using @default if default values are missing) and playground toggles
    • Checked Code Sandbox works for any docs examples
  • Code quality checklist
    • Added or updated jest tests
    • Updated visual regression tests
  • 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)
    • If the changes unblock an issue in a different repo, smoke tested carefully
  • Designer checklist
    • If applicable, file an issue to update EUI's Figma library with any corresponding UI changes

…lastic#8691)

Adds a new `fullHeight` prop to `EuiFlyoutBody` that applies flex layout
and full height to the inner overflow content container. This fixes
`EuiSelectable` (and other virtualized/height-dependent components) rendering
with a zero height when placed inside `EuiFlyoutBody`.

Closes elastic#8691
@ragini-pandey ragini-pandey requested a review from a team as a code owner February 26, 2026 04:33
@github-actions
Copy link

👋 Since this is a community submitted pull request, a Buildkite build has not been started automatically. Would an Elastic organization member please verify the contents of this pull request and kick off a build manually?

@github-actions github-actions bot added the community contribution (Don't delete - used for automation) label Feb 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community contribution (Don't delete - used for automation)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[EuiSelectable] height=full inside flyout with EuiFlyoutBody

1 participant