402744 - Notifier (new mail notification) is grey

Status: RESOLVED WONTFIX

(Thunderbird :: General, defect)

Description

[Pascal Herbert]

User-Agent:       Mozilla/5.0 (Windows; U; Windows NT 5.1; en-US; rv:1.9a9pre) Gecko/2007110605 Minefield/3.0a9pre
Build Identifier: version 3.0a1pre (2007110604)

In TB 2 the notifier's background for new messages is white, in the pre-versions it's grey. 

Reproducible: Always

Comment 1

[Magnus Melin [:mkmelin]]

While I can confirm it's grayish on linux too, technically it was never set to "white", but to " -moz-Dialog;" - maybe the default for that changed... 

http://lxr.mozilla.org/seamonkey/source/mail/themes/qute/mail/newmailalert.css#50

Comment 2

[David Ascher (:davida)]

not a blocker IMO

Comment 3

[Gary Kwong [:gkw] [:nth10sd] (NOT official MoCo now)]

(In reply to comment #1)
> While I can confirm it's grayish on linux too, technically it was never set to
> "white", but to " -moz-Dialog;" - maybe the default for that changed... 
> 
> http://lxr.mozilla.org/seamonkey/source/mail/themes/qute/mail/newmailalert.css#50
> 

Confirming based on above comment.

Comment 4

[Magnus Melin [:mkmelin]]

Actually my earlier comment isn't quite correct. That was changed from white to -moz-Dialog in bug 382299 which didn't land on branch.

Comment 5

[Paul F.]]

So the solution here is to change it back to white?
If possible I would like to work on it.

Comment 6

[Paul F.]]

More than just changing the background color I would like to bring this toast notification more up-to-date with today design standards.

Comment 7

[Paul F.]]

Last word, to avoid issues with high contrast color schemes we could add a check-box under the notification settings to turn on "high-contrast mode".

Comment 8

[Magnus Melin [:mkmelin]]

Actually, I think we should close this bug as is.

The solution is not to change it back to white. It's using -moz-Dialog done exactly so that if will follow system/theme colors for what you have installed. You shouldn't have to set any prefs for that.

However, if you want to improve the design, please submit a patch with your suggestion! But that's better done in a new bug.

Comment 9

[Paul F.]]

Sounds good. The good thing about the pref is that we don't have to compromise the looks of the UI because of a minority of users that will prefer to invert their screen colors, but I'll open a patch with the suggestion.
Thanks