Closed Bug 1275650 Opened 3 years ago Closed 3 years ago

Misleading #titlebar-buttonbox-container -moz-box-align in browser.css?

Categories

(Firefox :: Theme, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 49
Tracking Status
firefox49 --- fixed

People

(Reporter: mstange, Assigned: Gijs)

References

Details

Attachments

(1 file)

Bug 941058 added the following code to browser/base/content/browser.css:

/* Because these buttons don't move, they should always be aligned the same,
 * left and right were deprecated, so we have to do work to get it to mean that: */
#titlebar-buttonbox-container:-moz-locale-dir(ltr) {
  -moz-box-align: start;
}
#titlebar-buttonbox-container:-moz-locale-dir(rtl) {
  -moz-box-align: end;
}

But the XUL elements in the titlebar are:

vbox#titlebar
  hbox#titlebar-content
    spacer#titlebar-spacer
    hbox#titlebar-buttonbox-container
      hbox#titlebar-buttonbox
    hbox#titlebar-secondary-buttonbox

So the parent element of #titlebar-button-box-container is a hbox. -moz-box-align determines the alignment on the cross-axis, so in a hbox, it describes the *vertical* alignment.

So does this mean that the titlebar buttons shift down if your chrome is RTL?
(In reply to Markus Stange [:mstange] from comment #0)
> So does this mean that the titlebar buttons shift down if your chrome is RTL?

When I try this with force rtl, I don't see that happening, so I don't think so. I expect the code is just useless as-is. :-\
Assignee: nobody → gijskruitbosch+bugs
Status: NEW → ASSIGNED
Comment on attachment 8756582 [details]
MozReview Request: Bug 1275650 - remove useless CSS from content browser.css stylesheet, r?mikedeboer

https://reviewboard.mozilla.org/r/55262/#review52142

Remarkable!
Attachment #8756582 - Flags: review?(mdeboer) → review+
https://hg.mozilla.org/mozilla-central/rev/b8adc67d8f4a
Status: ASSIGNED → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 49
Depends on: 1280177
Depends on: 1296907
You need to log in before you can comment on or make changes to this bug.