1001869 - Use the new Australis toolbar button style for the AB on OS X.

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Josiah Bruner [:jsbruner]]

Attachment: newStyleForAB.patch

Bug 980491, but for the AB.

Comment 1

[Richard Marti (:Paenglab)]

Comment on attachment 8413220 [details] [diff] [review]
newStyleForAB.patch

Review of attachment 8413220 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good. The only difference to the main window buttons are the different margins and paddings. With this fixed r=me.

To fix the margin, please add this rule:
http://mxr.mozilla.org/comm-central/source/mail/themes/osx/mail/primaryToolbar.css#137
and remove line:
http://mxr.mozilla.org/comm-central/source/mail/themes/osx/mail/addrbook/addressbook.css#109

The padding change is below

Please could you also add the lines
http://mxr.mozilla.org/comm-central/source/mail/themes/osx/mail/primaryToolbar.css#103
to
http://mxr.mozilla.org/comm-central/source/mail/themes/osx/mail/primaryToolbar.css#117 ?

I know it isn't used in TB but a Add-on could use this and would look then correct.

::: mail/themes/osx/mail/addrbook/addressbook.css
@@ +30,4 @@
>  .toolbarbutton-1 > .toolbarbutton-menubutton-button,
>  .toolbarbutton-1 > .toolbarbutton-menubutton-dropmarker {
>    height: 22px;
> +  padding: 0px;

Please use:
padding: 0 4px;

Comment 2

[Josiah Bruner [:jsbruner]]

Attachment: Patch.

Addressed comments.

Comment 3

[Ryan VanderMeulen [:RyanVM]]

https://hg.mozilla.org/comm-central/rev/9de04a348c2c