@OwenEdwards fixed menu closing on ios, specifically ipad. closes vid…

…eojs#3158
OwenEdwards · Mar 7, 2016 · f82a3d1 · f82a3d1
1 parent 2105153
commit f82a3d1
Show file tree

Hide file tree

Showing 3 changed files with 7 additions and 1 deletion.
diff --git 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
@@ -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
@@ -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;