Skip to content
This repository has been archived by the owner on May 7, 2020. It is now read-only.

fix: color contrast a11y #133

Merged
merged 1 commit into from
Oct 28, 2019

Conversation

CDharmateja
Copy link
Contributor

Issue #42

Change color to maintain at least 4.5 color contrast

Changes

  • Color of Add to Contract

Flags

Related Issues

Signed-off-by: C Dharmateja <cdharmateja1999@gmail.com>
@irmerk irmerk added A11Y 🔓 Accessibility related Difficulty: Starter Hacktoberfest by DigitalOcean and DEV labels Oct 27, 2019
@irmerk irmerk added this to In Progress PR in Hacktoberfest 2019 via automation Oct 27, 2019
@irmerk irmerk added this to In Progress in Accessibility via automation Oct 27, 2019
Copy link
Contributor

@Michael-Grover Michael-Grover left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for taking this issue. Please change the color to #19C6C7

@Michael-Grover
Copy link
Contributor

@CDharmateja wait, is this changing the color of the border in between the buttons or the text?

image

If it's the border, that doesn't need be changed.

@irmerk
Copy link
Member

irmerk commented Oct 28, 2019

This is in conjunction with the PR in cicero-ui, I think the WCAG standards for this are more involved with text than borders.

@irmerk irmerk closed this Oct 28, 2019
Accessibility automation moved this from In Progress to Scrapped Oct 28, 2019
Hacktoberfest 2019 automation moved this from In Progress PR to Finished PR Oct 28, 2019
@irmerk irmerk moved this from Finished PR to Scrapped in Hacktoberfest 2019 Oct 28, 2019
@CDharmateja
Copy link
Contributor Author

The color is changing the color of the text "Add to contract" (not border).

@irmerk irmerk reopened this Oct 28, 2019
Accessibility automation moved this from Scrapped to In Progress Oct 28, 2019
Hacktoberfest 2019 automation moved this from Scrapped to In Progress PR Oct 28, 2019
@irmerk irmerk merged commit 324cc52 into accordproject:master Oct 28, 2019
Accessibility automation moved this from In Progress to Done Oct 28, 2019
Hacktoberfest 2019 automation moved this from In Progress PR to Finished PR Oct 28, 2019
@Michael-Grover
Copy link
Contributor

@CDharmateja please change the text color of "add to contract" to #19C6C7 so that it matches the "details" button

@CDharmateja
Copy link
Contributor Author

@Michael-Grover The pull request is merged now. What should I do?

@irmerk
Copy link
Member

irmerk commented Oct 28, 2019

We can change that later.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A11Y 🔓 Accessibility related Difficulty: Starter Hacktoberfest by DigitalOcean and DEV
Projects
Accessibility
  
Done
Hacktoberfest 2019
  
Finished PR
Development

Successfully merging this pull request may close these issues.

None yet

3 participants