From f3dd962d0cf13bd688babb46bce171a036f4e908 Mon Sep 17 00:00:00 2001 From: Conrad Chan Date: Fri, 31 Aug 2018 15:58:36 -0700 Subject: [PATCH] Fix: vertically center the CC button --- src/lib/viewers/media/Dash.scss | 2 +- src/lib/viewers/media/MediaControls.scss | 6 ++---- 2 files changed, 3 insertions(+), 5 deletions(-) diff --git a/src/lib/viewers/media/Dash.scss b/src/lib/viewers/media/Dash.scss index 0ef00f834..684decbfc 100644 --- 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 index a7e54bf4d..cfa44a026 100644 --- 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 {