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

Focused, unvisited links are (extremely) low-contrast #325

Closed
alex-ju opened this issue Sep 15, 2017 · 3 comments
Closed

Focused, unvisited links are (extremely) low-contrast #325

alex-ju opened this issue Sep 15, 2017 · 3 comments
Assignees

Comments

@alex-ju
Copy link

alex-ju commented Sep 15, 2017

An attempt to fix #264 raised this issue of having blue text on green buttons when focused on unvisited links.
screen shot 2017-09-15 at 14 21 40
screen shot 2017-09-15 at 14 21 53

Tested with latest Chrome, Firefox on OSX and Windows and IE9, IE10, IE11 on Windows.

If we do want to make focused links slightly darker to improve contrast, I would get rid of :link (which selects unvisited links). So the code below
a:link:focus { color: darken( $link-colour, 2.5%) }
would become
a:focus { color: darken( $link-colour, 2.5%) }
or even simpler, I would move the color attribution inside a:focus, which is a few lines above.

In this way we will have a "less powerful" selector (for improving contrast on links) that won't affect buttons.

If that sounds good, let me know and I'll come up with a PR.

@alex-ju alex-ju self-assigned this Sep 15, 2017
@NickColley
Copy link
Contributor

cc @edwardhorsford

@edwardhorsford
Copy link
Contributor

The button text should always be white. It's a mistake that it's blue.

@timpaul
Copy link
Contributor

timpaul commented Aug 30, 2018

Following the launch of the GOV.UK Design System, GOV.UK Template will now only get major bug fixes and security patches, so I'm going to close this.

@timpaul timpaul closed this as completed Aug 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants