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

Adds skip to content links #60. #69

Merged
merged 1 commit into from
Sep 9, 2020
Merged

Adds skip to content links #60. #69

merged 1 commit into from
Sep 9, 2020

Conversation

Lucas-Mc
Copy link
Collaborator

@Lucas-Mc Lucas-Mc commented Sep 9, 2020

Adds links at the top of each page to skip users directly to the content. This is useful for those using Lynx and Screen Readers as mentioned in #60. Fixes part of #60.

templates/header.html Outdated Show resolved Hide resolved
@tompollard
Copy link
Member

Are you confident that this is doing what we want to be doing? I'm not clear how these "skip to content" links are intended to be used, and I've not been able to test.

Note to self to check out these links:

@Lucas-Mc
Copy link
Collaborator Author

Lucas-Mc commented Sep 9, 2020

I used this link here as a guide for the CSS: https://webaim.org/techniques/css/invisiblecontent/ and I used the old website as a guide for the implementation. It has this:

<p class="u2">
<a href="#content" class="u2">Skip to content</a> |
<a href="#mainmenu" class="u2">Jump to navigation menu</a></p>

and then this later:

<a id="content"></a>

@tompollard
Copy link
Member

Okay, cool, if you're good with it then I'm happy. Please could you drop the CSS into the stylesheet? I'll then merge.

Adds links at the top of each page to skip users directly to the content. This is useful for those using Lynx and Screen Readers as mentioned in #60. Fixes part of #60.

Fixes link readability error

Adds CSS to stylesheet
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 this pull request may close these issues.

2 participants