Skip to content

[EuiPopover] Constrain popover dimensions to its container #6148

@chandlerprall

Description

@chandlerprall

Need to start with an investigation to better understand how the current sizing works (or doesn't), but a popover should limit its sizing to stay within its configured container (window by default).

After poking a little more into the guts of popover_positioning and EuiPopover, this is semi-interesting shenanigans as the positioning service doesn't actually ever set width on anything - it's deriving its calculations purely from the existing width of the panel content, so basically CSS itself is doing a confused psyduck once you set a % width on content popovers.

So, as Chandler mentioned in the above thread, we may need to start modifying EuiPopover to manually set width/height styles where we're currently not... but that may also cause other unexpected shenanigans with popover down the road. Fun times!

Originally posted by @constancecchen in #6146 (comment)

Metadata

Metadata

Assignees

No one assigned

    Labels

    stale-issue(Don't delete - used for automation)stale-issue-closed(Don't delete - used for automation)

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions