Closed Bug 382450 Opened 17 years ago Closed 17 years ago

new mail notification alert on linux does not respect system colors

Categories

(Thunderbird :: Mail Window Front End, defect)

x86
Linux
defect
Not set
minor

Tracking

(Not tracked)

RESOLVED DUPLICATE of bug 382299

People

(Reporter: patrick.bugs, Assigned: mscott)

Details

Attachments

(1 file)

User-Agent:       Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.3) Gecko/20061201 Firefox/2.0.0.3 (Ubuntu-feisty)
Build Identifier: version 2.0.0.0 (20070326)

I've only tested this problem on linux, so I don't know if it's persistent.

When new mail is received in Thunderbird, a little notification alert pops up.  This provides information about who sent the message, the first line of text, etc.  However, it also sets the CSS background color of the XUL widget without setting the CSS foreground color.  So, if you're using a white font on a dark background, you'll end up getting your normal white font, on a fixed white background.  In other words, invisible notifications.

In the case where a widget specifies a mandatory background color, it should also specify a foreground color.  I've corrected this by modifying chrome/skin/classic/messenger/newmailalert.css and chrome/skin/class/message/neMailAlertUnix.css to specify a black foreground color.


Reproducible: Always

Steps to Reproduce:
1. have white for default font color
2. wait for email
3. examine notification alert area
Actual Results:  
Most of the information in the notification area is invisible (white on white)

Expected Results:  
The information in the notification area should be readable.
Here's the patch that should fix the CSS problems for linux users who have a white font.
Could you attach your patch to bug 382299? Also, when do you, you should do a CVS diff (-up8 are good options to use) and ask for review from Scott MagGregor (mscott@moz). Thanks!
Status: UNCONFIRMED → RESOLVED
Closed: 17 years ago
Resolution: --- → DUPLICATE
patch posted, how do I request review?  It is not obvious from the UI or a few google searches.  For a person who is not a mozilla contributor, this is a bit of a drawback.
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: