Skip to content

Commit 606746b

Browse files
committed
fix: adjacent links to same location
skip the logo link by adding aria-hidden and tabindex -1 the 'home' link goes to the same location so this saves keyboard users a step
1 parent cd5dbd2 commit 606746b

File tree

1 file changed

+2
-1
lines changed

1 file changed

+2
-1
lines changed

_includes/nav.html

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,8 @@
33
<!-- <div class="container-fluid"> -->
44
<!-- Brand and toggle get grouped for better mobile display -->
55
<!-- <div class="navbar-header"> -->
6-
<a class="navbar-brand" aria-label="Code4Lib Home" href="{{ site.baseurl }}/"><img src="{{ "/assets/img/theme-images/c4l-logo.svg" | relative_url }}" alt="Code4Lib"></a>
6+
{% comment %} WAVE: avoid adjacent links to same URL by skipping this one & keeping "Home" below {% endcomment %}
7+
<a class="navbar-brand" aria-hidden="true" tabindex="-1" href="{{ site.baseurl }}/"><img src="{{ "/assets/img/theme-images/c4l-logo.svg" | relative_url }}" alt="Code4Lib"></a>
78
<!-- <button class="navbar-toggler" type="button" data-toggle="collapse" data-target="#bs-example-navbar-collapse-1" aria-controls="mainNavBar" aria-expanded="false" aria-label="Toggle navigation"> -->
89
<button class="navbar-toggler" type="button" data-toggle="collapse" data-target="#mainNavBar" aria-controls="mainNavBar" aria-expanded="false" aria-label="Toggle navigation">
910
<span class="sr-only">Toggle navigation</span>

0 commit comments

Comments
 (0)