From 4cf6871ccfeaf7a87eafad8298f8a701a2e63bf6 Mon Sep 17 00:00:00 2001
From: Marina Glancy <marina@moodle.com>
Date: Tue, 4 Apr 2017 12:30:18 +0800
Subject: [PATCH] MDL-57616 media_videojs: set min height for videos

This partially reverts changes from MDL-57187 that added the limit for all videos and not just videojs.
Videojs displays a big play button that overlaps contents below during video loading.
---
 filter/mediaplugin/styles.css   | 1 -
 media/player/videojs/styles.css | 4 ++++
 2 files changed, 4 insertions(+), 1 deletion(-)

diff --git a/filter/mediaplugin/styles.css b/filter/mediaplugin/styles.css
index fe08afaa68181..3c0a1a81f9ef3 100644
--- a/filter/mediaplugin/styles.css
+++ b/filter/mediaplugin/styles.css
@@ -14,7 +14,6 @@
     width: 100vw;
     max-width: 100%;
     height: auto;
-    min-height: 32px;
 }
 
 .mediaplugin > div {
diff --git a/media/player/videojs/styles.css b/media/player/videojs/styles.css
index 38e9e5934b029..7ab2e4a53d4c8 100644
--- a/media/player/videojs/styles.css
+++ b/media/player/videojs/styles.css
@@ -1330,3 +1330,7 @@ video::-webkit-media-text-track-display {
 .video-js.vjs-error {
   height: 150px;
 }
+/* Minimum height for videos should not be less than the size of play button. */
+.mediaplugin_videojs video {
+  min-height: 32px;
+}