Skip to content
This repository has been archived by the owner on Jan 9, 2019. It is now read-only.

Move heading text into anchor link #53

Open
LJWatson opened this issue Jul 23, 2016 · 0 comments
Open

Move heading text into anchor link #53

LJWatson opened this issue Jul 23, 2016 · 0 comments

Comments

@LJWatson
Copy link

The first <h2> on some pages contains two pieces of information: the "about" text and the "permalink" anchor. In some screen readers, this causes the heading to be broken down into its component parts.

<h2 class="col-xs-12">About <a href="#about" class="permalink">⚓</a></h2>

Suggest including "About" inside the anchor:

<h2 class="col-xs-12"><a href="#about" class="permalink">About ⚓</a></h2>

It would also be a good idea to hide the anchor symbol from screen readers. Screen readers will identify the link based on the <a>, making the anchor symbol a largely visual reference:

<h2 class="col-xs-12"><a href="#about" class="permalink">About <span aria-hidden="true">⚓</span></a></h2>

Note: aria-hidden should not be used to hide the anchor symbol, if the symbol is the only content of the link.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants