Skip to content
This repository has been archived by the owner on Sep 5, 2024. It is now read-only.

Commit

Permalink
demo(bottomSheet): add aria-labels for a11y
Browse files Browse the repository at this point in the history
Closes #572.
  • Loading branch information
ajoslin committed Nov 17, 2014
1 parent 2ece8cd commit 9adec2a
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<md-list>
<md-item ng-repeat="item in items" ng-click="listItemClick($index)">

<md-button class="md-grid-item-content">
<md-button class="md-grid-item-content" aria-label="{{item.name}}">
<div class="md-icon-container">
<md-inline-grid-icon icon="{{item.icon}}"></md-inline-grid-icon>
</div>
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@
<md-subheader>Comment Actions</md-subheader>
<md-list>
<md-item ng-repeat="item in items" ng-click="listItemClick($index)">
<md-button>
<md-button aria-label="{{item.name}}">
<!-- Using custom inline icon until md-icon is ready. DONT USE ME! -->
<md-inline-list-icon icon="{{item.icon}}"></md-inline-list-icon>
<span class="md-inline-list-icon-label">{{ item.name }}</span>
Expand Down

1 comment on commit 9adec2a

@marcysutton
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The aria-label is unnecessary here, those buttons already have text in them.

Please sign in to comment.