Skip to content

Commit

Permalink
fix: updates role to tab and moves aria-selected to li (#4451)
Browse files Browse the repository at this point in the history
  • Loading branch information
abbeyhrt authored and asudoh committed Oct 24, 2019
1 parent 0dd565e commit 9c354fa
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions packages/components/src/components/tabs/tabs.hbs
Original file line number Diff line number Diff line change
Expand Up @@ -14,9 +14,10 @@
{{#each items}}
<li
class="{{@root.prefix}}--tabs__nav-item{{#if selected}} {{@root.prefix}}--tabs__nav-item--selected{{/if}} {{#if disabled}} {{@root.prefix}}--tabs__nav-item--disabled {{/if}}"
data-target=".{{panelClass}}" role="presentation">
data-target=".{{panelClass}}" role="tab" {{#if selected}} aria-selected="true" {{/if}} {{#if disabled}}
aria-disabled="true" {{/if}}>
<a tabindex="0" id="{{linkId}}" class="{{@root.prefix}}--tabs__nav-link" href="javascript:void(0)" role="tab"
aria-controls="{{panelId}}" {{#if selected}} aria-selected="true" {{/if}}>{{label}}</a>
aria-controls="{{panelId}}">{{label}}</a>
</li>
{{/each}}
</ul>
Expand All @@ -29,4 +30,4 @@
<p style="padding: 0 1rem; font-size: 14px; font-weight: 600;">{{panelContent}}</p>
</div>
{{/each}}
</div>
</div>

0 comments on commit 9c354fa

Please sign in to comment.