Browse Source

Resolve issue https://github.com/owncast/owncast/issues/1044

- use built-in videojs font icon instead
- remove button clickhandler override to make button work on mobile
pull/1068/head
Ginger Wong 4 years ago
parent
commit
e89e65e1f5
  1. BIN
      webroot/img/video-settings.png
  2. 3
      webroot/js/components/player.js
  3. 7
      webroot/styles/video.css

BIN
webroot/img/video-settings.png

Binary file not shown.

Before

Width:  |  Height:  |  Size: 558 B

3
webroot/js/components/player.js

@ -168,8 +168,6 @@ class OwncastPlayer { @@ -168,8 +168,6 @@ class OwncastPlayer {
MenuButtonClass.call(this, player);
},
handleClick: function () {},
createItems: function () {
const defaultAutoItem = new MenuItem(player, {
selectable: true,
@ -224,7 +222,6 @@ class OwncastPlayer { @@ -224,7 +222,6 @@ class OwncastPlayer {
{},
player.controlBar.children_.length - 2
);
this.qualityMenuButton = menuButton;
}.bind(this)
);

7
webroot/styles/video.css

@ -14,8 +14,13 @@ video.video-js { @@ -14,8 +14,13 @@ video.video-js {
content: url("../img/airplay.png");
}
.vjs-quality-selector .vjs-icon-placeholder {
font-family: VideoJS;
font-weight: 400;
font-style: normal;
}
.vjs-quality-selector .vjs-icon-placeholder::before {
content: url("../img/video-settings.png");
content: "\f110";
}
.vjs-big-play-button {

Loading…
Cancel
Save