414590 - Move [mode="text"] toolbarbutton label styles to browser.css

Status: VERIFIED FIXED

(Firefox :: Toolbars and Customization, defect)

Description

[Stefan [:stefanh]]

Attachment: Move the (working) style rule to browser.css (no ui change)

toolkit/themes/pinstripe/global/toolbar.css:

toolbar[mode="text"] .toolbarbutton-1 > .toolbarbutton-text, toolbar[mode="text"] .toolbarbutton-1 > stack > toolbarbutton > .toolbarbutton-text {
  margin: 4px;
}

Those rules are pretty ancient and it seems that only the first one is actually applied. And I think the first rule fits better in browser.css. From what I see, Thunderbird is the only other consumer, and Thunderbird's toolbarbutton labels are messed up anyway in text mode. I'll fix thunderbird in a follow-up.

Comment 1

[Stefan [:stefanh]]

Comment on attachment 300041 [details] [diff] [review]
Move the (working) style rule to browser.css (no ui change)

Safe and simple move of a style rule.

Comment 2

[Mike Beltzner [:beltzner, not reading bugmail]]

Comment on attachment 300041 [details] [diff] [review]
Move the (working) style rule to browser.css (no ui change)

a=beltzner

Comment 3

[Stefan [:stefanh]]

Checking in toolkit/themes/pinstripe/global/toolbar.css;
/cvsroot/mozilla/toolkit/themes/pinstripe/global/toolbar.css,v  <--  toolbar.css
new revision: 1.15; previous revision: 1.14
done
Checking in browser/themes/pinstripe/browser/browser.css;
/cvsroot/mozilla/browser/themes/pinstripe/browser/browser.css,v  <--  browser.css
new revision: 1.113; previous revision: 1.112
done