Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue Labels color contrast #482

Closed
andrewmcgivery opened this issue Dec 13, 2014 · 15 comments
Closed

Issue Labels color contrast #482

andrewmcgivery opened this issue Dec 13, 2014 · 15 comments
Assignees

Comments

@andrewmcgivery
Copy link
Contributor

Many of the labels for issues do not have enough color contrast are are causing color contrast errors.

Here's a list of of labels that need some work... I have listed the current colors and colors that would pass AAA level color contrast. We could also go for AA level if we don't want to go all out. Up to you guys. I would suggest going for AA though.

Not sure if you guys want to go for these colors or do a further assessment. Some of these colors aren't the prettiest. :)

Also, it's a fairly big list so it make take a bit of work to solve all of these.

Label Current Color Passing Color (AAA Standard)
"Untriaged Issue" #6fc7fa #437d9e
"Needs Diagnosis" #ff8364 #b75c46
"Ready for Outreach" #a1ebbf #558066
"Closed" #799299 #657a80
"firefox" #FF4E00 #a42f00
"ie" #00B9EC #00617d
"mobile" #009800 #006800
"updated" #ededed #595959
"worksforme" #d4c5f9 #5c556e
"fixed" #009800 #006800
"ucweb" #f7c6c7 #6a5354
"iceweasel" #0052cc #0050c8
"ios" #0bd318 #026806
"sitewait" #006b75 #00636c
"contactready" #A1EBBF #3f604d
"android" #28ACBB #13626b
"safari" #1B9BF6 #0c5c95
"chrome" #F16A5A #943e34
"needsinfo" #fbca04 #6d5601
"invalid" #e6e6e6 #595959
"duplicate" #cccccc #595959

NOTE: "updated" label color also affects labels ie11, chromium, internet, nightly, and mozilla.

@miketaylr miketaylr added this to the Accessibility milestone Dec 17, 2014
@miketaylr
Copy link
Member

Let's tackle this and #436 at the same time.

@calexity
Copy link
Contributor

I think the most important ones to changes are the states of the bugs (new, needs diagnosis, etc). I don't think the labels need colors, so we can explore an option with no color differences or we can leave them as a decoration. We're getting color crazy anyway on this page:)

How about adding icons to the main labels like @karlcow suggested? Karl, I couldn't find all the ones you thought of, but here are some alts:

New - http://fortawesome.github.io/Font-Awesome/icon/random/
Needs diagnosis - http://fortawesome.github.io/Font-Awesome/icon/stethoscope/
Needs contact - http://fortawesome.github.io/Font-Awesome/icon/user/
Needs outreach (ready for outreach) - http://fortawesome.github.io/Font-Awesome/icon/paper-plane/ or http://fortawesome.github.io/Font-Awesome/icon/comments-o/
Waiting (site contacted) - http://fortawesome.github.io/Font-Awesome/icon/clock-o/
Closed - http://fortawesome.github.io/Font-Awesome/icon/check-square-o/

desktopicons

@andrewmcgivery check out the comments here and let me know what you think - #484 (comment)

@andrewmcgivery
Copy link
Contributor Author

From the other issue:

That all looks great. Awesome solutions there. I'd love to see this done in staging!

@miketaylr
Copy link
Member

Yeah, I like the icons as well.

I'm fine with ditching label colors @calexity -- maybe we just have a gray/whatever underline to indicate it's a clickable filter?

@calexity
Copy link
Contributor

calexity commented Jan 2, 2015

Yes! Our medium gray should be perfect.

On Fri, Jan 2, 2015, 11:13 AM Mike Taylor [email protected] wrote:

Yeah, I like the icons as well.

I'm fine with ditching label colors @calexity
https://github.com/calexity -- maybe we just have a gray/whatever
underline to indicate it's a clickable filter?


Reply to this email directly or view it on GitHub
#482 (comment)
.

@calexity
Copy link
Contributor

@karlcow thank you for pulling this one up!

It looks like we addressed the colors and we just haven't put them in the CSS. I would do that myself, but this app is far from the days when I could run it:)

  • New - (soft red) #ff8364 with black #000 text
  • Needs diagnosis - (gold) #efce82 with black #000 text
  • Needs contact - (light green) #b9edce with black #000 text
  • Needs outreach - (teal blue) #7abce1 with black #000 text
  • Waiting - (light blue) #b0daf0 with black #000 text
  • Closed - (gray) #c5d1d5 with black #000 text

See this comment for more: #484 (comment)

Is there something else I'm leaving unfinished here @karlcow?

@magsout
Copy link
Member

magsout commented Aug 14, 2015

@calexity

labels are already using these colors

:root {
  --wc-state-new: #ff8364; 
  --wc-state-need: #efce82; 
  --wc-state-needcontact: #b9edce; 
  --wc-state-ready: #7abce1; 
  --wc-state-sitewait: #b0daf0; 
  --wc-state-close: #c5d1d5; 
}

or maybe I missed someting ?

@magsout
Copy link
Member

magsout commented Oct 1, 2015

@andrewmcgivery how I can test if a color pass AAA level color contrast ?

@andrewmcgivery
Copy link
Contributor Author

I tend to use this tool:

http://webaim.org/resources/contrastchecker/

That help?

@magsout
Copy link
Member

magsout commented Oct 9, 2015

@andrewmcgivery oh thanks, sorry I didn't see your message

@magsout
Copy link
Member

magsout commented Oct 13, 2015

I tried the filters with colors proposed by @andrewmcgivery

Before:

capture d ecran 2015-10-13 a 22 58 46

After:

capture d ecran 2015-10-13 a 22 57 53

Filters are more visible. But I liked the progression of color before. @adamopenweb do you think there is a way to keep the old color but with good contrast ?

@miketaylr
Copy link
Member

I also prefer the before colors... does contrast improve if we change the text from white to black?

@andrewmcgivery
Copy link
Contributor Author

@miketaylr Changing white text to black will probably make most of the original colors pass actually.

@miketaylr
Copy link
Member

The current design is a little different, but when you hover or select the filter button there's still the low contrast white color. Let's change it to be bolder and maybe that'll pass.

@miketaylr
Copy link
Member

ping @magsout

@miketaylr miketaylr modified the milestones: London All Hands, Paris Meeting September 2015 Jun 14, 2016
magsout added a commit that referenced this issue Jun 14, 2016
magsout added a commit that referenced this issue Jun 14, 2016
magsout added a commit that referenced this issue Jun 14, 2016
magsout added a commit that referenced this issue Jun 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants