Attachment File Name is Hard to Read with Aero and a Dark Desktop Background

RESOLVED FIXED in Thunderbird 15.0

Status

Thunderbird
Message Reader UI
RESOLVED FIXED
5 years ago
5 years ago

People

(Reporter: mozilla, Assigned: Paenglab)

Tracking

11 Branch
Thunderbird 15.0
x86_64
Windows 7
Bug Flags:
in-testsuite -

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(4 attachments)

(Reporter)

Description

5 years ago
Created attachment 617174 [details]
attachment.png

User Agent: Mozilla/5.0 (Windows NT 6.1; WOW64; rv:11.0) Gecko/20100101 Firefox/11.0
Build ID: 20120312181643

Steps to reproduce:

Under Windows, enable Aero and set the desktop background to black.  Open an email with an attachment in Thunderbird 11.0.1 running the default theme.


Actual results:

I couldn't read the attachment's file name because the text was the same color as the background.


Expected results:

I should have been able to read the text because the text should have been a different color than the background.
(Assignee)

Comment 1

5 years ago
Created attachment 617208 [details] [diff] [review]
Patch

This patch removes the now needless -moz-win-borderless-glass appearance for the messageWindow and gives the attachmentBar the same color as in messengerWindow. Also styled the status-bar like in messengerWindow.
Assignee: nobody → richard.marti
Status: UNCONFIRMED → ASSIGNED
Ever confirmed: true
Attachment #617208 - Flags: ui-review?(bwinton)
Attachment #617208 - Flags: review?(bwinton)
(Assignee)

Comment 2

5 years ago
Created attachment 617648 [details]
Comparison before and after patch

This screenshot shows the attachment bars of standalone window and main window. On top before the patch, below with patch. The only difference is the color of the bar. With patch both have the same color, without patch not.
(Assignee)

Comment 3

5 years ago
Created attachment 617649 [details] [diff] [review]
Patch for aurora and beta

In case this patch should land on aurora or beta I've added a version without the status-bar color change for Australis.
Comment on attachment 617208 [details] [diff] [review]
Patch

I don't know if this is the same as the original reporter's problem, but it certainly does what it says in the patch description, so ui-r=me.

And the code seems fine, too, so r=me as well.

Thanks,
Blake.
Attachment #617208 - Flags: ui-review?(bwinton)
Attachment #617208 - Flags: ui-review+
Attachment #617208 - Flags: review?(bwinton)
Attachment #617208 - Flags: review+
(Assignee)

Updated

5 years ago
Keywords: checkin-needed
http://hg.mozilla.org/comm-central/rev/2f9a803068eb
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Flags: in-testsuite-
Keywords: checkin-needed
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 15.0
You need to log in before you can comment on or make changes to this bug.