From 2f080e8cd9ca7db5969b2c9968ae697cc3b58bc8 Mon Sep 17 00:00:00 2001 From: uzairr Date: Wed, 19 Feb 2020 04:21:17 +0500 Subject: [PATCH] Add five seconds step to video slider Currently, video is moving one second ahead or behind on each right and left arrow key respectively.In order to make it compliant with the industry standards like Youtube,its step is adjusted to five seconds. PROD-159 --- .../xmodule/js/spec/video/video_progress_slider_spec.js | 3 ++- .../xmodule/xmodule/js/src/video/06_video_progress_slider.js | 5 ++++- 2 files changed, 6 insertions(+), 2 deletions(-) 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 059ad032dd..0bf6ae39b1 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 @@ -30,7 +30,8 @@ min: 0, max: null, slide: state.videoProgressSlider.onSlide, - stop: state.videoProgressSlider.onStop + stop: state.videoProgressSlider.onStop, + step: 5 }); }); 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 af04bbe164..df7ac15e08 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 @@ -120,6 +120,7 @@ function() { edx.HtmlUtils.HTML('
') ); + // xss-lint: disable=javascript-jquery-append this.videoProgressSlider.el.append(sliderContents.text); this.videoProgressSlider.slider = this.videoProgressSlider.el @@ -128,7 +129,8 @@ function() { min: this.config.startTime, max: this.config.endTime, slide: this.videoProgressSlider.onSlide, - stop: this.videoProgressSlider.onStop + stop: this.videoProgressSlider.onStop, + step: 5 }); this.videoProgressSlider.sliderProgress = this.videoProgressSlider @@ -327,6 +329,7 @@ function() { msg = ngettext('%(value)s second', '%(value)s seconds', value); break; } + // xss-lint: disable=javascript-interpolate return interpolate(msg, {value: value}, true); };