diff --git a/CHANGELOG.md b/CHANGELOG.md index 5e28629a71..390335e012 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -8,6 +8,7 @@ CHANGELOG * @BrandonOCasey updated text track unit tests to use full es6 syntax ([view](https://github.com/videojs/video.js/pull/3148)) * @defli added missing var to sandbox index.html example ([view](https://github.com/videojs/video.js/pull/3155)) * @defli fixed typo and updated Turkish translations ([view](https://github.com/videojs/video.js/pull/3156)) +* @OwenEdwards fixed menu closing on ios, specifically ipad ([view](https://github.com/videojs/video.js/pull/3158)) -------------------- diff --git a/src/css/components/menu/_menu-popup.scss b/src/css/components/menu/_menu-popup.scss index 4387780e84..6eb4722b1a 100644 --- a/src/css/components/menu/_menu-popup.scss +++ b/src/css/components/menu/_menu-popup.scss @@ -19,7 +19,7 @@ max-height: 15em; } -.vjs-menu-button-popup:hover .vjs-menu, +.vjs-workinghover .vjs-menu-button-popup:hover .vjs-menu, .vjs-menu-button-popup .vjs-menu.vjs-lock-showing { display: block; } diff --git a/src/js/player.js b/src/js/player.js index b32003b3fb..ae72ca1136 100644 --- a/src/js/player.js +++ b/src/js/player.js @@ -196,6 +196,11 @@ class Player extends Component { // this.addClass('vjs-touch-enabled'); // } + // iOS Safari has broken hover handling + if (!browser.IS_IOS) { + this.addClass('vjs-workinghover'); + } + // Make player easily findable by ID Player.players[this.id_] = this;