Folder with new messages is yellow text on white - don't use "color: Highlight;"

RESOLVED FIXED in Thunderbird 3.3a1

Status

Thunderbird
Mail Window Front End
RESOLVED FIXED
8 years ago
7 years ago

People

(Reporter: BenB, Assigned: andreasn)

Tracking

Trunk
Thunderbird 3.3a1
All
Linux
Bug Flags:
blocking-thunderbird3 -
in-testsuite -

Firefox Tracking Flags

(blocking-thunderbird3.1 rc1+, thunderbird3.1 rc1-fixed)

Details

Attachments

(6 attachments)

(Reporter)

Description

8 years ago
Environment:
- SuSE 10.3
- KDE 3.x, with gtk-qt theme

Reproduction:
- Make your system theme so that selection (selected text in a textfield, or selected items in a listbox) is white text on turquoise background, or better yet black text on yellow background.
- Start Thunderbird, get new mail, look at folder pane

Actual result:
The folder with the new mail (not just unread, but really not seen yet) is bold and turquoise or yellow text on white background.
Hard to read, and really ugly.

Folders with unread, but not new mail are black text in bold. Until recently, folders with new mail also were like that.

Expected result:
Text color is readable and does not hurt in the eyes.
Normal black text (like all the other folder names), in bold, but with a different icon, e.g. the sparkl/splat as used in bug 516860 comment 26 for new messages in the thread pane.

Severity:
- Hurts eyes, hard to read
- No accessibility

Implementation:
Please don't use "color: Highlight" anywhere for anything. Highlight is a
background and border color. It may look good on your system/theme, but not on
mine: My Highlight color is a bright turquoise, which looks awful as text
color, and is hard to read.

FWIW, the correct (and common) usage of Highlight would be:
color: HighlightText;
background-color: Highlight;
but that's inappropriate in this case, because that's for selection.
That also shows the problem: If you use a very faint color as selection
background color (e.g. black on yellow), you get a very strange effect when
using that (yellow) as *foreground* color (with a white background).

Just use an icon.
(Reporter)

Updated

8 years ago
Flags: blocking-thunderbird3?
(Reporter)

Updated

8 years ago
Summary: Folder with new messages: don't use "color: Highlight;" → Folder with new messages is yellow text on white - don't use "color: Highlight;"
I was surprised to see how good, relatively speaking, the yellow on white on Linux looked last night when I was looking at the splat, since I'd only been seeing the use of Highlight in Windows, where I was wondering wtf we were thinking setting folders to the grey of disabled things to say that they were new. That was honestly my first thought the first time I saw it on Windows - "why is my Inbox _disabled_?"
We're not going to block the release on this.  However it would be good to get some work done on this.  Andreas, feel free to take this from me if you want.
Flags: blocking-thunderbird3? → blocking-thunderbird3-
Whiteboard: [no l10n impact]
(Reporter)

Updated

8 years ago
Assignee: clarkbw → ben.bucksch

Comment 3

8 years ago
Sigh, It looks horrible on ubuntu (orange on white)
(Reporter)

Comment 4

8 years ago
I just had some user in a web forum wondering why the folders are gray (for him).

Comment 5

8 years ago
I'd like to chime in that I miss the star/splat/whatever it was on the folder-view that indicated something new had arrived. Most of the time I have the folder-view in "Unread Folders"-mode. 
Each time I have 'new mail message' I can go see where that message was added by the filters as I have hundreds of unread messages spread among tens of folders and hence need a 'way' to see where the new messages went to. 
In tb2 the folder icon would receive a star/splat top right, but this seems to have gone. After a while I accidentally found out that the folders containing new messages in fact where of a different shade than those that did not receive anything. This difference is waayyy to subtle. 
I'm willing to attribute that this is partly because I use the 'classic windows' view (not vista's Wintendo look, nor Aero) with some modifications to the default back-ground colour and all that on a 1900x1200 TFT laptop screen that might not represent all colours perfectly.
I liked the way it worked on TB2, please either "revert" to that, or make the colour-scheme configurable. As it is now it's not really usable =(

Comment 6

8 years ago
I suggest to mark this bug as a duplicate of bug 533695.
Whiteboard: [no l10n impact] → [no l10n impact][UXprio]
(Assignee)

Comment 7

8 years ago
Created attachment 432819 [details] [diff] [review]
use "blue" instead of Highlight

Here is a patch using the color blue instead of Highlight.
(Assignee)

Comment 8

8 years ago
Created attachment 432820 [details]
screenshot
(Reporter)

Comment 9

8 years ago
Andreas, you don't know which background color the user uses on his desktop color scheme. There's no guarantee that it's something bright like white, it may just as well be black (and text white).

> Expected result:
> Text color is readable and does not hurt in the eyes.
> Normal black text (like all the other folder names), in bold, but with a
> different icon, e.g. the sparkl/splat as used in bug 516860 comment 26 for new
> messages in the thread pane.

Comment 10

8 years ago
Like Ben says, please don't use a fixed colour. Although I guess that most people stick to the OS-defaults, some of us prefer to use alternative colour-schemes.
Frankly I still prefer the splat-thing as I seem to remember it was in the previous version of Thunderbird...

(additionally I hope to buy a Notion Ink Adam (or similar) sometime which will actually be monochrome in some conditions, having the splat will then be ENORMOUSLY more useful than colours!)
(Assignee)

Comment 11

8 years ago
Sounds sane, I'll try to put together a patch with regular text colors and splats.
(Assignee)

Comment 12

8 years ago
Created attachment 433050 [details] [diff] [review]
patch using splat instead of color

Here is a version using the splat.
(Assignee)

Comment 13

8 years ago
Created attachment 433051 [details]
screenshot of the above patch
(Assignee)

Updated

8 years ago
Attachment #433050 - Attachment description: patch unsing splat instead of color → patch using splat instead of color
Attachment #433050 - Flags: ui-review?(clarkbw)
Attachment #433050 - Flags: review?(philringnalda)

Comment 14

8 years ago
Much more clear IMHO, thx !
Comment on attachment 433050 [details] [diff] [review]
patch using splat instead of color

splat it is
Attachment #433050 - Flags: ui-review?(clarkbw) → ui-review+
(Assignee)

Updated

8 years ago
Keywords: checkin-needed
(Assignee)

Comment 16

8 years ago
whups, sorry, no checkin-needed unless it got a code review
Keywords: checkin-needed
Comment on attachment 433050 [details] [diff] [review]
patch using splat instead of color

I'm pretty sure you don't want to wait while I sort out why my Linux VM has decided that building is something it only used to do, but: do you really mean "left" all those times, or do you actually mean "start" so it's not mangled in RTL?
Attachment #433050 - Flags: review?(philringnalda) → review?(mkmelin+mozilla)

Comment 18

8 years ago
This splat looks like a star, not a sun like next to messages (so we'd have bug 516860 again). It also covers the green arrow on the inbox in a not so nice way.
(Assignee)

Comment 19

8 years ago
(In reply to comment #18)
> This splat looks like a star, not a sun like next to messages (so we'd have bug
> 516860 again). It also covers the green arrow on the inbox in a not so nice
> way.

It's the same graphics we use in the message list, bug 516860 is about stars.

Comment 20

7 years ago
Comment on attachment 433050 [details] [diff] [review]
patch using splat instead of color

(In reply to comment #19)
> It's the same graphics we use in the message list.

Actually it isn't, that was my objection.

Minusing until that's sorted out
Attachment #433050 - Flags: review?(mkmelin+mozilla) → review-
We'd hold a few hours to get this fix, but not a day.  Reassigning to Andreas, since he's driving the patching here.
Assignee: ben.bucksch → nisses.mail
blocking-thunderbird3.1: --- → rc1+
(Assignee)

Comment 22

7 years ago
(In reply to comment #20)
> (From update of attachment 433050 [details] [diff] [review])
> (In reply to comment #19)
> > It's the same graphics we use in the message list.
> 
> Actually it isn't, that was my objection.

As I'm calling this piece of graphics as a background, I can't use -moz-image-region (yet [1]), so it needs to be a separate image.

1. http://www.css3.info/firefox-3-6-adds-background-clipping/
(Assignee)

Comment 23

7 years ago
Created attachment 445883 [details] [diff] [review]
attempt to use -moz-image-region on a background

also added some color to the text to match bug 562934
(Reporter)

Comment 24

7 years ago
> also added some color to the text to match bug 562934

I don't know about the qute theme, but in general, it's a bad idea to mix system colors (which gnomestripe uses) with absolute colors. You don't know what the system color is going to be: Background may be black, white, yellow, dark green, you name it. (E.g. Ubuntu has 2 themes: one with dark background and one with bright background.) The foreground needs to have appropriate contrast, and with an absolute color, you're pretty much guaranteed to stomp on some themes. In fact, that's pretty much what this bug is about. Or am I missing something in this particular case?
(Assignee)

Comment 25

7 years ago
Created attachment 445928 [details] [diff] [review]
the above patch without hardcoded colors
(Assignee)

Comment 26

7 years ago
(In reply to comment #24)
> > also added some color to the text to match bug 562934
> 
> I don't know about the qute theme, but in general, it's a bad idea to mix
> system colors (which gnomestripe uses) with absolute colors.

My guess is that cases where the color #0066cc is used as a background color is quite rare. The bug is about the fact that the systems Highlight color for some themes where producing odd results and that it wasn't a highlight. I would love for the GTK+ theming systems to give me more safe colors to play with, but that's a GTK+ bug.
Added another patch however.
(Reporter)

Comment 27

7 years ago
> that's a GTK+ bug.

No, it's inherent: A theme may use any color as background or foreground - that's the idea of themes. It's unlikely that they use exactly #0066cc, yes, but it's very likely that some of them use a background color which has little contrast to that, making the text unreadable.

> Added another patch however.

Thank you.
(Assignee)

Comment 28

7 years ago
Comment on attachment 433050 [details] [diff] [review]
patch using splat instead of color

Asking for a re-review based on the data in comment22. Will spin off a separate bug about using the moz-rect approach on the background image once the code base supports that so that we can use the same graphics for both.
Attachment #433050 - Flags: review- → review?(mkmelin+mozilla)
Comment on attachment 433050 [details] [diff] [review]
patch using splat instead of color

Stealing this review (sorry, Magnus!) so that we can get this in the tree by tonight's midnight freeze.
Attachment #433050 - Flags: review?(mkmelin+mozilla) → review?(dmose)
Given comment 22 (note that the link name itself is deceiving: apparently it _actually_ is only in 3.7 aka 1.9.3), I think this patch is Good Enough, so I'm giving it r+. Marking in-testsuite-, as I think the cost/benefit of writing an automated test for this patch is very low.
Flags: in-testsuite-

Updated

7 years ago
Attachment #433050 - Flags: review?(dmose) → review+
Pushed to the branch: <http://hg.mozilla.org/releases/comm-1.9.2/rev/9556406c5f63>.  The trunk tinderboxen are a bit of mess, so holding off on pushing to them for now.
status-thunderbird3.1: --- → rc1-fixed
Whiteboard: [no l10n impact][UXprio] → [pushed to branch; waiting for trunk greenery to push there]
Landed on trunk as well:

http://hg.mozilla.org/comm-central/rev/c6fe0eb353a7
Status: NEW → RESOLVED
Last Resolved: 7 years ago
Resolution: --- → FIXED
Whiteboard: [pushed to branch; waiting for trunk greenery to push there]
Target Milestone: --- → Thunderbird 3.2a1

Comment 33

7 years ago
@Ben Bucksch : on the contrary, your comment (#24) is exactly what (was/is) going wrong here... I found myself having trouble reading the "highlight coloured text" on a custom background (WinXP). I've switched to Win7 in the meanwhile and although I use a personalized 'classic' theme again, it now seems to be more readable... yet, only because I took the time to make Highlight more standing out towards the background colour, instead of 'just a couple of shades' away as before... 

I understand the developers 'running out of colours', but in that case why not implement a colour-options form and use whatever default you want but let the user tweak it as he sees fit. Might be a bit of a challenge across the different platforms, yet what's life without its little challenges ? =)

Anyway, looking forward to the next update...
(Reporter)

Comment 34

7 years ago
Roby, I think you misunderstood me. The problem you ran into is exactly what I wanted to prevent (I filed this bug).

Thanks, Andreas, for fixing this (and dmose for review/approval)!
You need to log in before you can comment on or make changes to this bug.