Skip to content

Commit

Permalink
split spacer into two files and converted to es6
Browse files Browse the repository at this point in the history
  • Loading branch information
mmcc committed Apr 23, 2015
1 parent 2160ddf commit 070290f
Show file tree
Hide file tree
Showing 5 changed files with 50 additions and 51 deletions.
3 changes: 2 additions & 1 deletion src/js/control-bar/control-bar.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,6 +17,7 @@ import ChaptersButton from './text-track-controls/chapters-button.js';
import SubtitlesButton from './text-track-controls/subtitles-button.js';
import CaptionsButton from './text-track-controls/captions-button.js';
import PlaybackRateMenuButton from './playback-rate-menu/playback-rate-menu-button.js';
import InsertionPointSpacer from './spacer-controls/insertion-point-spacer.js';

/**
* Container of main controls
Expand Down Expand Up @@ -44,7 +45,7 @@ ControlBar.prototype.options_ = {
'remainingTimeDisplay': {},
'liveDisplay': {},
'progressControl': {},
'insertionPointSeparator': {},
'insertionPointSpacer': {},
'fullscreenToggle': {},
'volumeControl': {},
'muteToggle': {},
Expand Down
2 changes: 1 addition & 1 deletion src/js/control-bar/progress-control/seek-handle.js
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ class SeekHandle extends SliderHandle {

/** @inheritDoc */
createEl() {
return super.createEl.call('div', {
return super.createEl('div', {
className: 'vjs-seek-handle',
'aria-live': 'off'
});
Expand Down
49 changes: 0 additions & 49 deletions src/js/control-bar/separator.js

This file was deleted.

23 changes: 23 additions & 0 deletions src/js/control-bar/spacer-controls/insertion-point-spacer.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
import Spacer from './spacer.js';

/**
* Spacer specifically meant to be used as an insertion point for new plugins, etc.
*
* @param {Player|Object} player
* @param {Obect=} options
*/
class InsertionPointSpacer extends Spacer {
buildCSSClass() {
return `vjs-insertion-point-spacer-control ${super.buildCSSClass}`;
}

createEl() {
return super.createEl({
className: this.buildCSSClass()
});
}
}

Spacer.registerComponent('InsertionPointSpacer', InsertionPointSpacer);

export default InsertionPointSpacer;
24 changes: 24 additions & 0 deletions src/js/control-bar/spacer-controls/spacer.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,24 @@
import Component from '../../component.js';

/**
* Just an empty spacer element that can be used as an append point for plugins, etc.
* Also can be used to create space between elements when necessary.
*
* @param {Player|Object} player
* @param {Object=} options
*/
class Spacer extends Component {
buildCSSClass() {
return `vjs-spacer-control ${super.buildCSSClass()}`;
}

createEl(props) {
return super.createEl('div', {
className: this.buildCSSClass()
});
}
}

Component.registerComponent('Spacer', Spacer);

export default Spacer;

0 comments on commit 070290f

Please sign in to comment.