Repository navigation
Bug - ModalFooter - Wrapping a Button in a ModalFooter with a Tooltip changes margins #12534
Description
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
Activity
Mayurpatel07 commentedon Jul 28, 2026
Hi! I'd like to work on this issue if it's available. I'll investigate the root cause and submit a fix.
Mayurpatel07 commentedon Jul 28, 2026
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?
github-actions commentedon Sep 27, 2026
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.