1085093 - Toolbar buttons lacks label margins

Status: RESOLVED FIXED

(SeaMonkey :: Themes, defect)

Description

[Stefan [:stefanh]]

https://hg.mozilla.org/mozilla-central/rev/e2b3ecf5916b from bug 1001871 introduced this change in toolbarbutton.css:

 .toolbarbutton-text {
-  margin: 0px;
+  margin: 0 !important; /* !important for overriding global.css */

This mean that any .toolbarbutton-text margin rule not having an !important will get overridden by the above rule. Also, the .toolbarbutton-text margin was over-ruled by the label margin rules in global.css before this change. Iotw, our large toolbar button now lacks label margins.

This can easily be seen in the PT and the main toolbars. For example, in the PT there's no space between the folder icon and the label.

Comment 1

[Stefan [:stefanh]]

I'm going to put up a quick fix for aurora/beta... but I think it might be worth improving the current look a bit.

Comment 2

[Stefan [:stefanh]]

Attachment: Restore margins

Too bad I didn't noticed this before... this will restore the margins. As I said, there is room for improvement, but it might be better to do that in a separate bug.

Comment 3

[Philip Chee]

Hmm why did they touch the theme files in toolkit if bug 1001871 is Firefox specific?

Comment 4

[Stefan [:stefanh]]

Comment on attachment 8507521 [details] [diff] [review]
Restore margins

[Approval Request Comment]
Regression caused by bug 1001871
User impact if declined: no margin between folders and labels in PT, not enough distance between nav-bar toolbarbuttons.
Testing completed (on m-c, etc.): Nope, but the fix is obvious
Risk to taking this patch (and alternatives if risky): Very low, it just restores previous behaviour
String changes made by this patch: none

Comment 5

[Stefan [:stefanh]]

http://hg.mozilla.org/comm-central/rev/50dd9f50d92b

Comment 6

[Stefan [:stefanh]]

http://hg.mozilla.org/releases/comm-aurora/rev/dc183e0df1fc
http://hg.mozilla.org/releases/comm-beta/rev/02b8a1d209a1