eui icon indicating copy to clipboard operation
eui copied to clipboard

[EuiPopover] New padding option between `s` and `m`?

Open MichaelMarcialis opened this issue 2 months ago • 3 comments

We don't currently offer a 12px padding option for our EuiPopover components or any of the subcomponents within (EuiPopoverTitle, EuiPopoverFooter). We have s (8px) and m (16px) sizes. Alone, this would be fine. However, there are many situations where our popovers contain other components where there is a padding mismatch (namely EuiSelectable) It's a minor thing, but something that has always bugged me.

To my knowledge, EuiSelectable only offers list items to have a horizontal padding of 0px or 12px. Ideally, the padding used in the EuiSelectable would be able to match the padding for the containing EuiPopover, EuiPopoverTitle, or EuiPopoverFooter, but that isn't presently possible. Should we add a new 12px padding option? And if so, how can we introduce it in a way that doesn't break things (given the small–medium naming convention)?

Alternatively, should we add more padding options for the EuiSelectable instead? Or change it's default 12px padding?

MichaelMarcialis avatar Oct 13 '25 20:10 MichaelMarcialis