Bug - ModalFooter - Wrapping a Button in a ModalFooter with a Tooltip changes margins

[mvollmer]

We have a button in a ModalFooter that might be disabled and then has a Tooltip to explain the reason. The Button has a wrong margin-inline-end style when it has a Tooltip.

Without theTooltip:

With the Tooltip:

You can see that the "Cancel" button has moved to the left a bit and is now touching the "Create and edit" button.

The code is something like this:

...
<ModalFooter>
  <Button>Create and run</Button>
  <Tooltip content={...}><Button isAriaDisabled>Create and edit</Button></Tooltip>
</ModalFooter>

There is some CSS somewhere that gives the Buttons in a ModalFooter a margin-inline-end value of 8px. This CSS however only applies to Buttons that are directly in a ModalFooter. It will be ignored for the Button wrapped in a Tooltip.

(Since the ModalFooter is a flex anyway, I would expect the button spacing to be handled by the column-gap of that, and not by margin manipulations, but maybe that doesn't work for some reason.)

---

Jira Issue: PF-4411

[Mayurpatel07]

Hi! I'd like to work on this issue if it's available. I'll investigate the root cause and submit a fix.

[Mayurpatel07]

Hi! I investigated this issue and reproduced it locally.

The root cause is the direct-child selector used for spacing:

.pf-v6-c-modal-box__footer > .pf-v6-c-button:not(:last-child)

When a Button is wrapped in Tooltip, it is no longer a direct child of ModalFooter, so the spacing rule no longer applies.

I checked existing PatternFly layout patterns (for example ActionList), and they use container-owned gap/column-gap instead of child margins.

My proposed fix is to replace the direct-child button spacing with container-owned column-gap on ModalFooter. Does that approach sound acceptable before I open a PR?