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

Commit bbb719d

Browse files
EladBezalelandrewseguin
authored andcommitted
fix(slider): role was applied to inner element instead of the directive itself (#10731)
fixes #10565
1 parent 7256670 commit bbb719d

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/components/slider/slider.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ function SliderDirective($$rAF, $window, $mdAria, $mdUtil, $mdConstant, $mdThemi
172172

173173
if (tAttrs.disabled || tAttrs.ngDisabled) wrapper.attr('tabindex', -1);
174174

175-
wrapper.attr('role', 'slider');
175+
tElement.attr('role', 'slider');
176176

177177
$mdAria.expect(tElement, 'aria-label');
178178

src/components/slider/slider.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -190,7 +190,7 @@ describe('md-slider', function() {
190190

191191
pageScope.$apply('model = 102');
192192

193-
expect(wrapper.attr('role')).toEqual('slider');
193+
expect(slider.attr('role')).toEqual('slider');
194194
expect(slider.attr('aria-valuemin')).toEqual('100');
195195
expect(slider.attr('aria-valuemax')).toEqual('104');
196196
expect(slider.attr('aria-valuenow')).toEqual('102');

0 commit comments

Comments
 (0)