Skip to content

Commit e12d020

Browse files
bradfrostbmuenzenmeyer
authored andcommitted
Merge pull request #13 from bradfrost/feature/add-aria-to-dropdown-lists
Added aria-expanded to dropdown navigation
1 parent bc10471 commit e12d020

File tree

3 files changed

+12
-2
lines changed

3 files changed

+12
-2
lines changed

packages/docs/src/_includes/components/header.njk

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,9 +10,12 @@
1010

1111
<div class="c-header__nav-container js-nav-panel">
1212
{% include "components/tree-nav.njk" %}
13+
14+
15+
1316
</div>
1417
</div>
1518
<!--end c-header__inner-->
1619
</header>
17-
<!--end c-header-->
20+
<!--end c-header-->
1821

packages/docs/src/_includes/components/tree-nav.njk

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,12 @@
55
{% for item in navigation.items %}
66
<li class="c-tree-nav__item {% if item.subnav %}js-nav-dropdown{% endif %}">
77
{% if item.subnav %}
8-
<button class="c-tree-nav__link c-tree-nav__link--btn js-nav-dropdown-trigger">
8+
<button class="c-tree-nav__link c-tree-nav__link--btn js-nav-dropdown-trigger" aria-expanded="true">
99
{{ item.label }}
1010
{% set className = "c-tree-nav__icon" %}
1111
{% include "components/icon-chevron-down.njk" %}
12+
13+
1214

1315
</button>
1416

packages/docs/src/js/primary-nav.js

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,15 @@
2323
if (navLinkParent.classList.contains('is-active')) {
2424
/* 3 */
2525
navLinkParent.classList.remove('is-active');
26+
27+
this.setAttribute("aria-expanded", "false");
2628
} else {
2729
/* 4 */
2830
navLinkParent.classList.add('is-active');
31+
32+
this.setAttribute("aria-expanded", "true");
2933
}
34+
3035
});
3136
}
3237

0 commit comments

Comments
 (0)