From 708d4ad901053cc7e337c454cc393abf3e0bcf99 Mon Sep 17 00:00:00 2001 From: Carey Hinoki Date: Thu, 29 Sep 2016 16:36:23 -0700 Subject: [PATCH] style(_progress): Provide padding for the `vjs-progress-holder` while allowing it to resize on hover. Thereby containing the `MouseTimeDisplay` to the `vjs-progress-holder` instead of it bleeding outside. NOTE: Guessing that the margin was put in place to provide some spacing between the slider and the surrounding controls. Because the slider itself, or rather, `vjs-progress-holder` becomes larger on hover, we cannot have the "padding" also be responsive on the same element. --- src/css/components/_progress.scss | 1 + src/css/components/_slider.scss | 1 - 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/src/css/components/_progress.scss b/src/css/components/_progress.scss index f684b7cf57..8b5cc45225 100644 --- a/src/css/components/_progress.scss +++ b/src/css/components/_progress.scss @@ -15,6 +15,7 @@ .video-js .vjs-progress-control { @include flex(auto); @include display-flex(center); + margin: 0 0.45em 0 0.45em; min-width: 4em; } diff --git a/src/css/components/_slider.scss b/src/css/components/_slider.scss index bbf2bd99b4..5d5e758a1a 100644 --- a/src/css/components/_slider.scss +++ b/src/css/components/_slider.scss @@ -3,7 +3,6 @@ position: relative; cursor: pointer; padding: 0; - margin: 0 0.45em 0 0.45em; @include background-color-with-alpha($secondary-background-color, $secondary-background-transparency); }