Bug 487632 - Editor line numbers do not have enough color contrast
Summary: Editor line numbers do not have enough color contrast
Status: RESOLVED FIXED
Alias: None
Product: Orion (Archived)
Classification: ECD
Component: Editor (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P2 normal (vote)
Target Milestone: ---   Edit
Assignee: Carolyn MacLeod CLA
QA Contact:
URL:
Whiteboard:
Keywords: accessibility
Depends on:
Blocks:
 
Reported: 2016-02-10 19:36 EST by Carolyn MacLeod CLA
Modified: 2016-03-07 12:45 EST (History)
2 users (show)

See Also:


Attachments
orion-editor-line-numbers.png (9.71 KB, image/png)
2016-02-10 19:36 EST, Carolyn MacLeod CLA
no flags Details

Note You need to log in before you can comment on or make changes to this bug.
Description Carolyn MacLeod CLA 2016-02-10 19:36:17 EST
Created attachment 259696 [details]
orion-editor-line-numbers.png

The line numbers in the editor do not have enough color contrast to meet the WCAG 2.0 AA standard.

Their current color is silver on white, or rgb(192, 192, 192) on rgb(255, 255, 255).

In order to be WCAG 2.0 AA compliant, the minimum color the numbers need to be is
rgb(118, 118, 118)  (aka #767676).

This "slightly darker silver" color looks very similar to the current silver.

In the attached screen snapshot, the numbers 5 and 7 have been changed to be WCAG 2.0 AA compliant.
Comment 1 Carolyn MacLeod CLA 2016-02-19 16:59:43 EST
One of only 2 remaining problems before we can say that the Orion Editor passes WCAG 2.0 AA for color contrast.
Comment 2 Steve Northover CLA 2016-02-20 03:06:25 EST
Please fix it.  Get SSQ to approve the look and paste the image back here (or just release it when the code freeze is over and tell me).
Comment 3 Carolyn MacLeod CLA 2016-03-05 08:40:32 EST
How do I clear the settings cache on the server?
I can't figure out how to test my changes - the old colors always win.

Anton, how did you force changes out to everyone after changing a color in a default theme (i.e. Prospecto)? You must know of a nice hack for this?  :)
Comment 4 Silenio Quarti CLA 2016-03-07 10:08:19 EST
You need to increment THEMES_VERSION in ThemeVersion.js to force everyone to get the new theme definition.
Comment 5 Carolyn MacLeod CLA 2016-03-07 11:08:56 EST
Thank-you!! I was banging my head against a wall on that one. That fixes it nicely. Releasing to Gerrit - SSQ please have a look when it comes through.
Note that it's important that the demo have the new contrast color also.
Note that the (old?) color #CCCCCC does not have enough contrast either.
Comment 6 Eclipse Genie CLA 2016-03-07 11:13:08 EST
New Gerrit change created: https://git.eclipse.org/r/67905
Comment 8 Carolyn MacLeod CLA 2016-03-07 12:45:59 EST
Fixed with commit in comment 7.