Skip to content

Commit

Permalink
Do not show the outline on the skip link target container
Browse files Browse the repository at this point in the history
* Putting it in style.css solves the issue for our specific skip link, but leaves the outline working on other anchors in case developers need that.
* For discussion, see: #814
* Related: #755 and #604

Fixes #543
  • Loading branch information
davidakennedy committed Nov 11, 2015
1 parent e1fe1fc commit 1a7cedc
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 0 deletions.
5 changes: 5 additions & 0 deletions sass/modules/_accessibility.scss
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -25,3 +25,8 @@
z-index: 100000; /* Above WP toolbar. */ z-index: 100000; /* Above WP toolbar. */
} }
} }

/* Do not show the outline on the skip link target. */
#content[tabindex="-1"]:focus {
outline: 0;
}
5 changes: 5 additions & 0 deletions style.css
Original file line number Original file line Diff line number Diff line change
Expand Up @@ -664,6 +664,11 @@ a:active {
z-index: 100000; /* Above WP toolbar. */ z-index: 100000; /* Above WP toolbar. */
} }


/* Do not show the outline on the skip link target. */
#content[tabindex="-1"]:focus {
outline: 0;
}

/*-------------------------------------------------------------- /*--------------------------------------------------------------
# Alignments # Alignments
--------------------------------------------------------------*/ --------------------------------------------------------------*/
Expand Down

0 comments on commit 1a7cedc

Please sign in to comment.