v5.8.2 dist

CHANGELOG.md

@@ -6,6 +6,9 @@ _(none)_
 
 --------------------
 
+## 5.8.2 (2016-03-09)
+* @gkatsev fixed chapters menu. Fixes #3062 ([view](https://github.com/videojs/video.js/pull/3163))
+
 ## 5.8.1 (2016-03-07)
 * @gkatsev updated videojs badges in the README ([view](https://github.com/videojs/video.js/pull/3134))
 * @BrandonOCasey converted remaining text-track modules to ES6 ([view](https://github.com/videojs/video.js/pull/3130))

component.json

@@ -1,7 +1,7 @@
 {
   "name": "video.js",
   "description": "An HTML5 and Flash video player with a common API and skin for both.",
-  "version": "5.8.1",
+  "version": "5.8.2",
   "keywords": [
     "videojs",
     "html5",

dist/alt/video-js-cdn.css

@@ -1128,10 +1128,6 @@ video::-webkit-media-text-track-display {
   100% {
     border-top-color: #73859f; } }
 
-.vjs-chapters-button .vjs-menu {
-  left: -10em;
-  width: 0; }
-
 .vjs-chapters-button .vjs-menu ul {
   width: 24em; }
 

dist/alt/video.novtt.js

@@ -1,6 +1,6 @@
 /**
  * @license
- * Video.js 5.8.1 <http://videojs.com/>
+ * Video.js 5.8.2 <http://videojs.com/>
  * Copyright Brightcove, Inc. <https://www.brightcove.com/>
  * Available under Apache License Version 2.0
  * <https://github.com/videojs/video.js/blob/master/LICENSE>
@@ -6411,11 +6411,13 @@ var ChaptersButton = (function (_TextTrackButton) {
     var menu = this.menu;
     if (menu === undefined) {
       menu = new _menuMenuJs2['default'](this.player_);
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      var title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: _utilsToTitleCaseJs2['default'](this.kind_),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     if (chaptersTrack && chaptersTrack.cues == null) {
@@ -8355,11 +8357,13 @@ var MenuButton = (function (_ClickableComponent) {
 
     // Add a title list item to the top
     if (this.options_.title) {
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      var title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: _utilsToTitleCaseJs2['default'](this.options_.title),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     this.items = this['createItems']();
@@ -19772,7 +19776,7 @@ setup.autoSetupTimeout(1, videojs);
  *
  * @type {String}
  */
-videojs.VERSION = '5.8.1';
+videojs.VERSION = '5.8.2';
 
 /**
  * The global options object. These are the settings that take effect

dist/video-js.css

@@ -9,9 +9,6 @@
 .video-js .vjs-big-play-button:before, .video-js .vjs-control:before {
   text-align: center; }
 
-.video-js {
-  color: blue; }
-
 @font-face {
   font-family: VideoJS;
   src: url("font/VideoJS.eot?#iefix") format("eot"); }
@@ -1131,10 +1128,6 @@ video::-webkit-media-text-track-display {
   100% {
     border-top-color: #73859f; } }
 
-.vjs-chapters-button .vjs-menu {
-  left: -10em;
-  width: 0; }
-
 .vjs-chapters-button .vjs-menu ul {
   width: 24em; }
 

dist/video.js

@@ -1,6 +1,6 @@
 /**
  * @license
- * Video.js 5.8.1 <http://videojs.com/>
+ * Video.js 5.8.2 <http://videojs.com/>
  * Copyright Brightcove, Inc. <https://www.brightcove.com/>
  * Available under Apache License Version 2.0
  * <https://github.com/videojs/video.js/blob/master/LICENSE>
@@ -6415,11 +6415,13 @@ var ChaptersButton = (function (_TextTrackButton) {
     var menu = this.menu;
     if (menu === undefined) {
       menu = new _menuMenuJs2['default'](this.player_);
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      var title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: _utilsToTitleCaseJs2['default'](this.kind_),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     if (chaptersTrack && chaptersTrack.cues == null) {
@@ -8359,11 +8361,13 @@ var MenuButton = (function (_ClickableComponent) {
 
     // Add a title list item to the top
     if (this.options_.title) {
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      var title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: _utilsToTitleCaseJs2['default'](this.options_.title),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     this.items = this['createItems']();
@@ -19776,7 +19780,7 @@ setup.autoSetupTimeout(1, videojs);
  *
  * @type {String}
  */
-videojs.VERSION = '5.8.1';
+videojs.VERSION = '5.8.2';
 
 /**
  * The global options object. These are the settings that take effect

package.json

@@ -1,7 +1,7 @@
 {
   "name": "video.js",
   "description": "An HTML5 and Flash video player with a common API and skin for both.",
-  "version": "5.8.1",
+  "version": "5.8.2",
   "copyright": "Copyright Brightcove, Inc. <https://www.brightcove.com/>",
   "license": "Apache-2.0",
   "keywords": [

src/css/components/_chapters.scss

@@ -2,11 +2,6 @@
   @extend .vjs-icon-chapters;
 }
 
-.vjs-chapters-button .vjs-menu {
-  left: -10em; // (Width of vjs-menu - width of vjs-control) / 2
-  width: 0;
-}
-
 .vjs-chapters-button .vjs-menu ul {
   width: 24em;
 }

src/js/control-bar/text-track-controls/chapters-button.js

@@ -90,11 +90,13 @@ class ChaptersButton extends TextTrackButton {
     let menu = this.menu;
     if (menu === undefined) {
       menu = new Menu(this.player_);
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      let title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: toTitleCase(this.kind_),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     if (chaptersTrack && chaptersTrack.cues == null) {

src/js/menu/menu-button.js

@@ -70,11 +70,13 @@ class MenuButton extends ClickableComponent {
 
     // Add a title list item to the top
     if (this.options_.title) {
-      menu.contentEl().appendChild(Dom.createEl('li', {
+      let title = Dom.createEl('li', {
         className: 'vjs-menu-title',
         innerHTML: toTitleCase(this.options_.title),
         tabIndex: -1
-      }));
+      });
+      menu.children_.unshift(title);
+      Dom.insertElFirst(title, menu.contentEl());
     }
 
     this.items = this['createItems']();