Skip to content
This repository has been archived by the owner on Jan 17, 2023. It is now read-only.

Insufficient color contrast on server pages #5014

Closed
kimberlythegeek opened this issue Oct 8, 2018 · 4 comments
Closed

Insufficient color contrast on server pages #5014

kimberlythegeek opened this issue Oct 8, 2018 · 4 comments
Labels
a11y good first issue server wontfix Closed (without being fixed) because we are discontinuing the server

Comments

@kimberlythegeek
Copy link
Contributor

Element has insufficient color contrast of 3
foreground color: #009ec0,
background color: #f9f9fa,
font size: 9.0pt,
font weight: normal

Expected contrast ratio of 4.5:1

image

detailed information on color-contrast rule

Rule Violated:
  color-contrast - Ensures the contrast between foreground and background colors meets WCAG 2 AA contrast ratio thresholds
  	URL: https://dequeuniversity.com/rules/axe/3.1/color-contrast?application=axeAPI
  	Impact Level: serious
  	Tags: cat.color wcag2aa wcag143
  	Elements Affected:
  	1) Target: .legal-links > a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(1)
  		Element has insufficient color contrast of 3 (foreground color: #009ec0, background color: #f9f9fa, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
  	2) Target: a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(2)
  		Element has insufficient color contrast of 3 (foreground color: #009ec0, background color: #f9f9fa, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
  	3) Target: a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(3)
  		Element has insufficient color contrast of 3 (foreground color: #009ec0, background color: #f9f9fa, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
  	4) Target: a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(4)
  		Element has insufficient color contrast of 3 (foreground color: #009ec0, background color: #f9f9fa, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
  	5) Target: a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(5)
  		Element has insufficient color contrast of 3 (). Expected contrast ratio of 4.5:1
  	6) Target: a[target="_blank"][rel="noopener\ noreferrer"]:nth-child(6)
  		Element has insufficient color contrast of 3 (foreground color: #009ec0, background color: #f9f9fa, font size: 9.0pt, font weight: normal). Expected contrast ratio of 4.5:1
@kimberlythegeek
Copy link
Contributor Author

Also, I don't have sufficient permissions to add tags or request reviews.

@johngruen johngruen added the a11y label Oct 9, 2018
@punamdahiya punamdahiya added this to the Sprint 24 (65-1) milestone Oct 10, 2018
@ianb ianb modified the milestones: Sprint 24 (65-1), Sprint 25 (65-2) 🎃 Oct 29, 2018
@johngruen johngruen modified the milestones: Sprint 25 (65-2) 🎃, Sprint 26 (65-3) Nov 12, 2018
@punamdahiya
Copy link
Contributor

With new UI, Insufficient color contrast on footer links is resolved. Using this issue to resolve color contrast error seen on homepage,shots and myshots page

@punamdahiya punamdahiya changed the title Insufficient color contrast on footer links Insufficient color contrast on server pages Nov 29, 2018
@punamdahiya
Copy link
Contributor

punamdahiya commented Nov 30, 2018

Running homepage screenshots.firefox.com with axe extension errors insufficient color contrast for
a) text inside banner (.message > p)
b) white text 'Firefo Screenshots Screenshots made simple. Take, save, and share screenshots without leaving Firefox.' on background-color #04D1E6 as contrast ratio is less than 3.

Changing text-color to black over background #04D1E6 increases contrast ratio to 11 and fixes error.

@eeejay @kimberlythegeek whats general guideline for making page color contrast accessible, will ensuring contrast in few sections depending on text usability suffice.

I am seeing color contrast accessibility error while analyzing few major websites with axe so not sure if we should blanket fix all contrast errors.

@youwenliang @johngruen To fix all the contrast errors we will need to request UX feedback to provide new foreground- background color and images.

@ianb ianb added the server wontfix Closed (without being fixed) because we are discontinuing the server label Feb 7, 2019
@ianb
Copy link
Contributor

ianb commented Feb 7, 2019

This issue is being closed (without being fixed) because we are discontinuing the Screenshots server.

@ianb ianb closed this as completed Feb 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11y good first issue server wontfix Closed (without being fixed) because we are discontinuing the server
Projects
None yet
Development

No branches or pull requests

5 participants