web/video.js
1
0
Fork 0
mirror of https://github.com/videojs/video.js.git synced 2024-11-28 08:58:46 +02:00
Code Releases Activity

Peg the volume control to 1.0 on init

Browse Source 
Setup styles so that the volume control initially renders at full volume. This matches browser behavior where volume is available and saves Javascript from having to manually update the volume control on init. After the initial draw, the volume control is updated dynamically the same way it was before.
This commit is contained in:
David LaPalomento 2014-03-31 16:22:43 -04:00
parent 8bc861fb86
commit d5957da3b2
2 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/css/video-js.less
View File

@ -303,6 +303,7 @@ fonts to show/hide properly.
top: 0;
left: 0;
height: 0.5em;
width: 100%;
background: @slider-bar-color
url(@slider-bar-pattern)
@ -311,6 +312,10 @@ fonts to show/hide properly.
.vjs-default-skin .vjs-volume-bar .vjs-volume-handle {
width: 0.5em;
height: 0.5em;
/* Assumes volume starts at 1.0. If you change the size of the
handle relative to the volume bar, you'll need to update this value
too. */
left: 90%;
}
.vjs-default-skin .vjs-volume-handle:before {

1
src/js/control-bar/volume-control.js
View File

@ -49,7 +49,6 @@ vjs.VolumeBar = vjs.Slider.extend({
vjs.Slider.call(this, player, options);
player.on('volumechange', vjs.bind(this, this.updateARIAAttributes));
player.ready(vjs.bind(this, this.updateARIAAttributes));
setTimeout(vjs.bind(this, this.update), 0); // update when elements is in DOM
}
});