Open Bug 1722946 Opened 4 years ago Updated 2 years ago

Stop using -4px margins on inline dialog margins in commonDialog and friends, and update all the children to account for this

Categories

(Toolkit :: Content Prompts, defect)

Desktop
All
defect

Tracking

()

Tracking Status
firefox92 --- affected

People

(Reporter: Gijs, Unassigned)

Details

commonDialog and a few other places use a 16px grid padding, but then subtract 4px which gets "added back" by all the inline items in the dialog, cf. https://searchfox.org/mozilla-central/rev/cfc40681e13089f92fb3f5f67d5d527fb04d9505/toolkit/components/prompts/content/commonDialog.css#103-105 .

In bug 1720262 I ended up doing a little dance to mimic this, but it ends up being more effort than just defining the 16px (or at least, it feels like that).

It would be nice if fixing this was straightforward, but IME it is tricky because of the button box in the dialog, whose margins are tricky to affect from outside of the anonymous content of the dialog.

I'm not 100% confident what the "right" solution here looks like, but the current stuff is a bit of a fudge.

Severity: -- → S4
Component: Notifications and Alerts → Content Prompts
You need to log in before you can comment on or make changes to this bug.