Skip to content
This repository was archived by the owner on May 29, 2019. It is now read-only.

Commit da71159

Browse files
Evan.Duncanwesleycho
Evan.Duncan
authored andcommitted
fix(carousel): improve accessibility
- Improves accessibility of carousel Closes #4478 Closes #4479
1 parent 19118f1 commit da71159

File tree

1 file changed

+15
-7
lines changed

1 file changed

+15
-7
lines changed

template/carousel/carousel.html

+15-7
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,16 @@
11
<div ng-mouseenter="pause()" ng-mouseleave="play()" class="carousel" ng-swipe-right="prev()" ng-swipe-left="next()">
2-
<ol class="carousel-indicators" ng-show="slides.length > 1">
3-
<li ng-repeat="slide in slides | orderBy:indexOfSlide track by $index" ng-class="{active: isActive(slide)}" ng-click="select(slide)"></li>
4-
</ol>
5-
<div class="carousel-inner" ng-transclude></div>
6-
<a class="left carousel-control" ng-click="prev()" ng-show="slides.length > 1"><span class="glyphicon glyphicon-chevron-left"></span></a>
7-
<a class="right carousel-control" ng-click="next()" ng-show="slides.length > 1"><span class="glyphicon glyphicon-chevron-right"></span></a>
8-
</div>
2+
<div class="carousel-inner" ng-transclude></div>
3+
<a role="button" href="#" class="left carousel-control" ng-click="prev()" ng-show="slides.length > 1">
4+
<span aria-hidden="true" class="glyphicon glyphicon-chevron-left"></span>
5+
<span class="sr-only">previous</span>
6+
</a>
7+
<a role="button" href="#" class="right carousel-control" ng-click="next()" ng-show="slides.length > 1">
8+
<span aria-hidden="true" class="glyphicon glyphicon-chevron-right"></span>
9+
<span class="sr-only">next</span>
10+
</a>
11+
<ol class="carousel-indicators" ng-show="slides.length > 1">
12+
<li ng-repeat="slide in slides | orderBy:indexOfSlide track by $index" ng-class="{ active: isActive(slide) }" ng-click="select(slide)">
13+
<span class="sr-only">slide {{ $index + 1 }} of {{ slides.length }}<span ng-if="isActive(slide)">, currently active</span></span>
14+
</li>
15+
</ol>
16+
</div>

0 commit comments

Comments
 (0)