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.
ragusa87 · Apr 7, 2017 · 4cf6871 · 4cf6871
1 parent bd99cb9
commit 4cf6871
Show file tree

Hide file tree

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