1
0
mirror of https://github.com/videojs/video.js.git synced 2025-03-17 21:18:27 +02:00

fix: add buildWrapperCSSClass methods to all menu buttons (#4147)

In #4034, we changed the way that menu buttons work slightly by introduction a wrapper element with a separate wrapper css builder. However, this broke, at least the playback-rate menu.
This PR adds a buildWrapperCSSClass to each of our menu buttons.
This commit is contained in:
Gary Katsevman 2017-03-02 15:04:35 -05:00 committed by GitHub
parent a9f8fcb2a2
commit 61d427c7ee
5 changed files with 18 additions and 0 deletions

View File

@ -62,6 +62,10 @@ class PlaybackRateMenuButton extends MenuButton {
return `vjs-playback-rate ${super.buildCSSClass()}`;
}
buildWrapperCSSClass() {
return `vjs-playback-rate ${super.buildWrapperCSSClass()}`;
}
/**
* Create the playback rate menu
*

View File

@ -38,6 +38,10 @@ class CaptionsButton extends TextTrackButton {
return `vjs-captions-button ${super.buildCSSClass()}`;
}
buildWrapperCSSClass() {
return `vjs-captions-button ${super.buildWrapperCSSClass()}`;
}
/**
* Update caption menu items
*

View File

@ -41,6 +41,10 @@ class ChaptersButton extends TextTrackButton {
return `vjs-chapters-button ${super.buildCSSClass()}`;
}
buildWrapperCSSClass() {
return `vjs-chapters-button ${super.buildWrapperCSSClass()}`;
}
/**
* Update the menu based on the current state of its items.
*

View File

@ -76,6 +76,9 @@ class DescriptionsButton extends TextTrackButton {
return `vjs-descriptions-button ${super.buildCSSClass()}`;
}
buildWrapperCSSClass() {
return `vjs-descriptions-button ${super.buildWrapperCSSClass()}`;
}
}
/**

View File

@ -37,6 +37,9 @@ class SubtitlesButton extends TextTrackButton {
return `vjs-subtitles-button ${super.buildCSSClass()}`;
}
buildWrapperCSSClass() {
return `vjs-subtitles-button ${super.buildWrapperCSSClass()}`;
}
}
/**