1400583 - Font smoothing background color is now a CSS property (Bug 1387594)

Status: RESOLVED FIXED

(Thunderbird :: Theme, defect)

Description

[Stefan [:stefanh]]

In order to get the right font smoothing background color in vibrant areas you need to use the CSS property "-moz-font-smoothing-background-color". I haven't checked, but it's probably just the source lists that needs it.

Comment 1

[Richard Marti (:Paenglab)]

Thank you, Stefan. I have already a patch and I hope you aren't on creating one.

Comment 2

[Richard Marti (:Paenglab)]

Attachment: fontSmoothing.patch

Unfortunately we have no Mac reviewer. So please could you review it?

Comment 3

[Stefan [:stefanh]]

Comment on attachment 8909016 [details] [diff] [review]
fontSmoothing.patch

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

I'm usually called stefanh (commit msg) ;-)

In messengercompose.css there’s one occurrence of ’-moz-appearance: menupopup;’ and one occurrence of ’-moz-appearance: menuitem;’, but I assume the toolkit files takes care of that. But then the -moz-appearance rules could be removed from messengercompose.css (there's no -moz-font-smoothing-background-color rule for https://dxr.mozilla.org/comm-central/rev/bda524beac249b64aa36016800502a34073bf35a/mozilla/toolkit/themes/osx/global/menu.css#10 and I've asked mstange about that). 

You might want to fix im/themes/conversation.css as well (there's one 'moz-mac-source-list' rule there).

Comment 4

[Richard Marti (:Paenglab)]

Attachment: fontSmoothing.patch

Thanks

(In reply to Stefan [:stefanh] from comment #3)
> Comment on attachment 8909016 [details] [diff] [review]
> fontSmoothing.patch
> 
> Review of attachment 8909016 [details] [diff] [review]:
> -----------------------------------------------------------------
> 
> I'm usually called stefanh (commit msg) ;-)

Sorry, fixed.

> In messengercompose.css there’s one occurrence of ’-moz-appearance:
> menupopup;’ and one occurrence of ’-moz-appearance: menuitem;’, but I assume
> the toolkit files takes care of that. But then the -moz-appearance rules
> could be removed from messengercompose.css (there's no
> -moz-font-smoothing-background-color rule for
> https://dxr.mozilla.org/comm-central/rev/
> bda524beac249b64aa36016800502a34073bf35a/mozilla/toolkit/themes/osx/global/
> menu.css#10 and I've asked mstange about that). 

I checked it and the toolkit rules for font-smoothing are active. The -moz-appearance rules are needed because toolkit sets them to 'none' with the menulist[editable="true"] > ... rule and I want the normal menupopup appearance.

> You might want to fix im/themes/conversation.css as well (there's one
> 'moz-mac-source-list' rule there).

This is for IB and I don't build it for checking if it is working correctly.

Comment 5

[Jorg K (CEST = GMT+2)]

Shouldn't we add it to im/themes/conversation.css anyway? We usually keep IB going with all the other ports we do if they're easy.

Comment 6

[Richard Marti (:Paenglab)]

Attachment: fontSmoothing.patch

Okay, I'm convinced. Added the IB file.

Comment 7

[Jorg K (CEST = GMT+2)]

https://hg.mozilla.org/comm-central/rev/937beb54bc99a4ebce47bf01d2aaebee9d6636e0
Font smoothing background color is now a CSS property (Bug 1387594). r=stefanh

In the last patch the "h" was missing from the reviewer's name, but nothing escapes the good sheriff ;-)