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

Add skip to main content link #51

Merged
merged 1 commit into from
Sep 20, 2019
Merged

Conversation

dsamojlenko
Copy link
Member

@dsamojlenko dsamojlenko commented Sep 20, 2019

image

@timarney timarney temporarily deployed to cds-node-starter-pr-51 September 20, 2019 18:25 Inactive
@timarney
Copy link
Member

If you rebase this it should pass the tests.

@timarney
Copy link
Member

Let's merge this - wondering why the skip link doesn't have a focus outline but we can chat with @JuliannaR next week.

@dsamojlenko
Copy link
Member Author

dsamojlenko commented Sep 20, 2019

I'm assuming the focus outline is just not visible because I set the background to the same yellow... so we can adjust. Like I mentioned in comment above, mostly just stole styles from Gov.UK for initial implementation ... we can make it our own in collaboration with Julianna for sure.

@dsamojlenko dsamojlenko merged commit eb08ffd into master Sep 20, 2019
@dsamojlenko dsamojlenko deleted the add_skip_to_main_content_link branch September 20, 2019 18:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants