Skip to content

Commit 2f0834f

Browse files
chrisbousteadgkatsev
authored andcommitted
fix(menu-button): make menu button title a component (videojs#5722)
Fixes videojs#3612, fixes videojs#5759
1 parent 4b11a4e commit 2f0834f

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

src/js/menu/menu-button.js

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -108,16 +108,17 @@ class MenuButton extends Component {
108108

109109
// Add a title list item to the top
110110
if (this.options_.title) {
111-
const title = Dom.createEl('li', {
111+
const titleEl = Dom.createEl('li', {
112112
className: 'vjs-menu-title',
113113
innerHTML: toTitleCase(this.options_.title),
114114
tabIndex: -1
115115
});
116116

117117
this.hideThreshold_ += 1;
118118

119-
menu.children_.unshift(title);
120-
Dom.prependTo(title, menu.contentEl());
119+
const titleComponent = new Component(this.player_, {el: titleEl});
120+
121+
menu.addItem(titleComponent);
121122
}
122123

123124
this.items = this.createItems();

0 commit comments

Comments
 (0)