Bug 75128

Comment 1 Tod Creasey 2005-08-08 10:32:42 EDT

Dejan what is the status of this?

Comment 2 Dejan Glozic 2005-08-08 10:51:26 EDT

We didn't look at it yet. Do you consider it bad enough for 3.1.1?

Comment 3 Karice McIntyre 2005-08-24 13:44:39 EDT

I just did a quick check in 3.1 and 3.2 (post M1).  I can see the text in the 
cheat sheets.  I will attach a pic.  We may be able to close this one.

Comment 4 Karice McIntyre 2005-08-24 13:45:58 EDT

Created attachment 26415 [details]
pic of cheat sheet in high contrast

Comment 5 Chris Goldthorpe 2006-03-23 20:54:00 EST

Created attachment 36860 [details]
Another screenshot, this one is hard to read

I just tried this and the results were not very easy to read. See also Bug 132573 which proposes that the background color for the active step be a neutral color.

Comment 6 Chris Goldthorpe 2006-04-04 19:53:32 EDT

Created attachment 37698 [details]
Proposed Patch

This patch uses a better algorithm for determining the background colors when in a mode with dark background. For light backgrounds (i.e. almost all users) the color scheme is not changed by this patch.

Comment 7 Chris Goldthorpe 2006-04-04 19:56:08 EDT

Created attachment 37699 [details]
Screenshot with patch applied

Note that the hyperlink still shows up as dark, this color comes from the SWT pallette and can be reset in the preferences.

Comment 8 Curtis d'Entremont 2006-04-05 11:58:44 EDT

I applied the patch, as it seems to help in some color schemes. As for high-contrast mode, it doesn't change anything as far as I can tell. The only problem I see in high contrast is the title, it's a bit dark. I'm guessing this color is handled by forms - can you confirm Chris?

Comment 9 Curtis d'Entremont 2006-04-05 11:59:25 EDT

Chris, see my last comment.

Comment 10 Curtis d'Entremont 2006-04-05 11:59:54 EDT

Created attachment 37761 [details]
screenshot in high contrast after patch

Comment 11 Chris Goldthorpe 2006-04-05 17:12:39 EDT

On a high contrast setting with a dark background the patch will always improve readability because the old code contained these lines which caused the background to be closer to white than black.

// colored background
rgb = toolkit.getColors().getSystemColor(SWT.COLOR_LIST_SELECTION);
// active color is selection + 70% white
rgb = FormColors.blend(rgb, white, 30);

On a high contrast setting with a white background the background is a light shade of blue, that is not affected by this patch. White would be a better background choice for high contrast white, this is addressed in Bug 132573 .

There is still a problem with the title color in high contrast, which comes from UI Forms (see Bug 135152 ).

To summarize the situation is much improved and cheat sheets are now usable on a high contrast screen but we should probably keep the bug open until the title color is fixed. 

Comment 12 Curtis d'Entremont 2006-06-14 13:56:30 EDT

Sending to Platform UA inbox, either because owner is incorrect or no longer works on UA.

Comment 13 Karice McIntyre 2006-12-04 09:47:54 EST

UA team, what is the status/target of this bug?

Comment 14 Chris Goldthorpe 2006-12-04 11:49:03 EST

Targeted for 3.3.

Comment 15 Chris Goldthorpe 2007-01-30 18:51:16 EST

The title color is now fixed in forms so I will close this one.