Fix: vertically center the CC button

box · Aug 31, 2018 · f3dd962 · f3dd962
1 parent a6e84f3
commit f3dd962
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/src/lib/viewers/media/Dash.scss b/src/lib/viewers/media/Dash.scss
@@ -4,9 +4,9 @@
     video {
         cursor: none;
         display: block;
-        min-width: 460px;
         max-height: 100%;
         max-width: 100%;
+        min-width: 460px;
         width: 100%;
     }
 

diff --git a/src/lib/viewers/media/MediaControls.scss b/src/lib/viewers/media/MediaControls.scss
@@ -165,16 +165,14 @@
 .bp-media-controls-cc-icon-text {
     background-color: rgba($black, .3); // Anything less than .3 looks too transparent on IE/Edge
     border-radius: 4px;
-    bottom: 2px;
     color: $white;
     display: inline-block;
     font-size: 12px;
     font-weight: 600;
-    height: 22px;
+    height: 24px;
     letter-spacing: .1em;
-    line-height: 22px;
+    line-height: 24px;
     padding: 0 0 0 1px; // Padding left 1px because text isn't perfectly centered on Safari/Firefox even with text-align:center. Looks fine with extra-padding on Chrome/IE/Edge
-    position: relative; // For lining up the bottom with other buttons in the controls bar
     width: 24px;
 
     &:hover {