web/video.js
1
0
Fork 0
mirror of https://github.com/videojs/video.js.git synced 2025-01-27 11:22:06 +02:00
Code Releases Activity

fix(icon-placeholder): align icons on ie8 properly (#4174)

Browse Source 
Make sure that the button inside the menu button element isn't a
vjs-control but make sure that the icon-placeholder is sized properly.
This commit is contained in:
Gary Katsevman 2017-03-07 15:43:44 -05:00 committed by GitHub
parent 05e64948c5
commit 1770f00018
3 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
src/css/components/_control.scss
View File

@ -10,12 +10,12 @@
width: 4em;
@include flex(none);
& > .vjs-icon-placeholder:before {
font-size: 1.8em;
line-height: 1.67;
}
.vjs-button > .vjs-icon-placeholder:before {
font-size: 1.8em;
line-height: 1.67;
@extend %icon-default;
}
@extend %icon-default;
}
// Replacement for focus outline

1
src/js/control-bar/text-track-controls/subs-caps-button.js
View File

@ -23,7 +23,6 @@ class SubsCapsButton extends TextTrackButton {
this.label_ = 'captions';
}
this.menuButton_.controlText(toTitleCase(this.label_));
}
/**

1
src/js/menu/menu-button.js
View File

@ -38,6 +38,7 @@ class MenuButton extends Component {
const buttonClass = Button.prototype.buildCSSClass();
this.menuButton_.el_.className = this.buildCSSClass() + ' ' + buttonClass;
this.menuButton_.removeClass('vjs-control');
this.addChild(this.menuButton_);