AC-407 adding lable to video progress

This commit is contained in:
Chris Rodriguez
2016-04-29 12:58:16 -04:00
parent bf9380523f
commit 721fffc9c4
3 changed files with 5 additions and 3 deletions

View File

@@ -44,7 +44,7 @@
expect(timeControl).toHaveAttrs({
'role': 'slider',
'title': 'Video position',
'aria-label': 'Video position',
'aria-disabled': 'false'
});

View File

@@ -83,7 +83,6 @@ function () {
// handle, behaves as a slider named 'video position'.
state.videoProgressSlider.handle.attr({
'role': 'slider',
'title': gettext('Video position'),
'aria-disabled': false,
'aria-valuetext': getTimeDescription(state.videoProgressSlider
.slider.slider('option', 'value')),
@@ -91,6 +90,7 @@ function () {
'aria-valuemin': '0',
'aria-valuenow': state.videoPlayer.currentTime,
'tabindex': '0'
'aria-label': gettext('Video position')
});
state.el.on('destroy', state.videoProgressSlider.destroy);

View File

@@ -50,7 +50,9 @@ function() {
'</span>',
'</span>',
'</button>',
'<div class="volume-slider-container" aria-hidden="true" title="' + gettext('Adjust video volume') + '">', // jshint ignore: line
'<div class="volume-slider-container" aria-hidden="true" title="',
gettext('Adjust video volume'),
'">',
'<div class="volume-slider" ',
'role="slider"',
'aria-orientation="vertical" ',