From 204b52e92dc8371579d9e18457b3001fcc775b20 Mon Sep 17 00:00:00 2001 From: Waheed Ahmed Date: Fri, 17 Oct 2014 19:32:44 +0500 Subject: [PATCH] Added three missing required states for video progress slider. TNL-89 --- .../js/spec/video/video_progress_slider_spec.js | 5 +++++ .../xmodule/js/src/video/06_video_progress_slider.js | 11 ++++++++++- 2 files changed, 15 insertions(+), 1 deletion(-) diff --git a/common/lib/xmodule/xmodule/js/spec/video/video_progress_slider_spec.js b/common/lib/xmodule/xmodule/js/spec/video/video_progress_slider_spec.js index 32dcdc915f..629b6e3833 100644 --- a/common/lib/xmodule/xmodule/js/spec/video/video_progress_slider_spec.js +++ b/common/lib/xmodule/xmodule/js/spec/video/video_progress_slider_spec.js @@ -122,6 +122,11 @@ 'option', 'value', 20 ); }); + + it('required aria values updated', function () { + expect(state.videoProgressSlider.handle.attr('aria-valuenow')).toBe('20'); + expect(state.videoProgressSlider.handle.attr('aria-valuemax')).toBe('120'); + }); }); }); diff --git a/common/lib/xmodule/xmodule/js/src/video/06_video_progress_slider.js b/common/lib/xmodule/xmodule/js/src/video/06_video_progress_slider.js index aa506dc25c..2e20df0fbe 100644 --- a/common/lib/xmodule/xmodule/js/src/video/06_video_progress_slider.js +++ b/common/lib/xmodule/xmodule/js/src/video/06_video_progress_slider.js @@ -76,7 +76,10 @@ function () { 'title': gettext('Video position'), 'aria-disabled': false, 'aria-valuetext': getTimeDescription(state.videoProgressSlider - .slider.slider('option', 'value')) + .slider.slider('option', 'value')), + 'aria-valuemax': state.videoPlayer.duration(), + 'aria-valuemin': '0', + 'aria-valuenow': state.videoPlayer.currentTime }); } @@ -243,6 +246,12 @@ function () { .slider('option', 'max', duration) .slider('option', 'value', time); } + + // Update aria values. + this.videoProgressSlider.handle.attr({ + 'aria-valuemax': duration, + 'aria-valuenow': time + }); } // When the video stops playing (either because the end was reached, or