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

c-typecyan does not meet some accessibility requirements. #233

Open
varjmes opened this issue Mar 1, 2017 · 4 comments
Open

c-typecyan does not meet some accessibility requirements. #233

varjmes opened this issue Mar 1, 2017 · 4 comments

Comments

@varjmes
Copy link
Contributor

varjmes commented Mar 1, 2017

I can go around fixing all the uses of it, but our c-typecyan - #008fe9 colour does not pass accessibility requirements against a white background in any case apart from large text (WCAG AA).

What's the scope to remove it / ask people not use it on white backgrounds?

image

@varjmes varjmes changed the title c-typecyan does not meet accessibility requirements. c-typecyan does not meet some accessibility requirements. Mar 1, 2017
@annez
Copy link
Contributor

annez commented Mar 1, 2017

I think we need to approach this from a what's the scope to tweak the shade rather then a removal or a set of rules for usage. In this scenario if we were to set rules, the amount of work to move towards adhering to that is much more complicated than triggering a ustyle upgrade notice.

We were aware of this when we picked the colour but it was close enough to "accessible" we decided to run with it. In the current state we'd have to revisit the palette - c-typecyan is used quite a bit.

Maybe the right step in this scenario is to outline the biggest culprits of the usage of this colour in an inaccessible manner?

@varjmes
Copy link
Contributor Author

varjmes commented Mar 1, 2017

I think we need to approach this from a what's the scope to tweak the shade

Sounds good to me.

Maybe the right step in this scenario is to outline the biggest culprits of the usage of this colour in an inaccessible manner?

Do you mean get a list of places where we use this and try and fix the worst / most visible / most important cases?

@annez
Copy link
Contributor

annez commented Mar 1, 2017

Do you mean get a list of places where we use this and try and fix the worst / most visible / most important cases?

Yes, but rather than try and fix them, we should understand their usage first. A search through GitHub for color: $c-typecyan should suffice to begin with. Maybe we should start with the worst culprits within ustyle.

@varjmes
Copy link
Contributor Author

varjmes commented Mar 1, 2017

Good shout! I'll add that to my list of things to do.

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

2 participants