diff --git a/src/css/components/_big-play.scss b/src/css/components/_big-play.scss index ad458a667b..c23dc339df 100644 --- a/src/css/components/_big-play.scss +++ b/src/css/components/_big-play.scss @@ -12,7 +12,7 @@ opacity: 1; border: 0.06666em solid $primary-foreground-color; - /* Need a slightly gray bg so it can be seen on black backgrounds */ + // Need a slightly gray bg so it can be seen on black backgrounds @include background-color-with-alpha($primary-background-color, $primary-background-transparency); @include border-radius(0.3em); @include transition(all 0.4s); diff --git a/src/css/components/_chapters.scss b/src/css/components/_chapters.scss index a20628a9cd..b70b018ee0 100644 --- a/src/css/components/_chapters.scss +++ b/src/css/components/_chapters.scss @@ -3,7 +3,7 @@ } .vjs-chapters-button .vjs-menu { - left: -10em; /* (Width of vjs-menu - width of vjs-control) / 2 */ + left: -10em; // (Width of vjs-menu - width of vjs-control) / 2 width: 0; } diff --git a/src/css/components/_control-bar.scss b/src/css/components/_control-bar.scss index 588bd7cc96..be09029cb8 100644 --- a/src/css/components/_control-bar.scss +++ b/src/css/components/_control-bar.scss @@ -32,7 +32,7 @@ .vjs-controls-disabled .vjs-control-bar, .vjs-using-native-controls .vjs-control-bar, .vjs-error .vjs-control-bar { - /* !important is ok in this context. */ + // !important is ok in this context. display: none !important; } @@ -42,17 +42,16 @@ visibility: visible; } -/* IE8 is flakey with fonts, and you have to change the actual content to force -fonts to show/hide properly. -- "\9" IE8 hack didn't work for this -- Found in XP IE8 from http://modern.ie. Does not show up in "IE8 mode" in IE9 -*/ +// IE8 is flakey with fonts, and you have to change the actual content to force +// fonts to show/hide properly. +// - "\9" IE8 hack didn't work for this +// Found in XP IE8 from http://modern.ie. Does not show up in "IE8 mode" in IE9 $ie8screen: "\0screen"; .vjs-user-inactive.vjs-playing .vjs-control-bar :before { @media #{$ie8screen} { content: ""; } } -/* IE 8 + 9 Support */ +// IE 8 + 9 Support .vjs-has-started.vjs-no-flex .vjs-control-bar { display: table; } diff --git a/src/css/components/_control.scss b/src/css/components/_control.scss index b3506b9f57..2246b2ae40 100644 --- a/src/css/components/_control.scss +++ b/src/css/components/_control.scss @@ -19,19 +19,19 @@ } } -/* Replacement for focus outline */ +// Replacement for focus outline .video-js .vjs-control:focus:before, .video-js .vjs-control:hover:before, .video-js .vjs-control:focus { text-shadow: 0em 0em 1em rgba($primary-foreground-color, 1); } -/* Hide control text visually, but have it available for screenreaders */ +// Hide control text visually, but have it available for screenreaders .video-js .vjs-control-text { @include hide-visually; } -/* IE 8 + 9 Support */ +// IE 8 + 9 Support .vjs-no-flex .vjs-control { display: table-cell; vertical-align: middle; diff --git a/src/css/components/_fullscreen.scss b/src/css/components/_fullscreen.scss index c36a9c00c1..f23f6748ad 100644 --- a/src/css/components/_fullscreen.scss +++ b/src/css/components/_fullscreen.scss @@ -6,7 +6,7 @@ .video-js .vjs-fullscreen-control { @extend .vjs-icon-fullscreen-enter; } -/* Switch to the exit icon when the player is in fullscreen */ +// Switch to the exit icon when the player is in fullscreen .video-js.vjs-fullscreen .vjs-fullscreen-control { @extend .vjs-icon-fullscreen-exit; } diff --git a/src/css/components/_layout.scss b/src/css/components/_layout.scss index a496e0e0de..14f2c2f52e 100644 --- a/src/css/components/_layout.scss +++ b/src/css/components/_layout.scss @@ -1,9 +1,8 @@ .video-js { - /* display:inline-block would be closer to the video el's display:inline - * but it results in flash reloading when going into fullscreen [#2205] - */ + // display:inline-block would be closer to the video el's display:inline + // but it results in flash reloading when going into fullscreen [#2205] display: block; - /* Make video.js videos align top when next to video elements */ + // Make video.js videos align top when next to video elements vertical-align: top; box-sizing: border-box; @@ -11,21 +10,21 @@ background-color: #000; position: relative; padding: 0; - /* Start with 10px for base font size so other dimensions can be em based and - easily calculable. */ + // Start with 10px for base font size so other dimensions can be em based and + // easily calculable. font-size: 10px; line-height: 1; - /* Provide some basic defaults for fonts */ + // Provide some basic defaults for fonts font-weight: normal; font-style: normal; - /* Avoiding helvetica: issue #376 */ + // Avoiding helvetica: issue #376 font-family: $text-font-family; @include user-select(none); - /* Fix for Firefox 9 fullscreen (only if it is enabled). Not needed when - checking fullScreenEnabled. */ + // Fix for Firefox 9 fullscreen (only if it is enabled). Not needed when + // checking fullScreenEnabled. &:-moz-full-screen { position: absolute; } &:-webkit-full-screen { @@ -34,35 +33,35 @@ } } -/* All elements inherit border-box sizing */ +// All elements inherit border-box sizing .video-js *, .video-js *:before, .video-js *:after { box-sizing: inherit; } -/* List style reset */ +// List style reset .video-js ul { font-family: inherit; font-size: inherit; line-height: inherit; list-style-position: outside; - /* Important to specify each */ + // Important to specify each margin-left: 0; margin-right: 0; margin-top: 0; margin-bottom: 0; } -/* Fill the width of the containing element and use padding to create the - desired aspect ratio. Default to 16x9 unless another ratio is given. */ +// Fill the width of the containing element and use padding to create the +// desired aspect ratio. Default to 16x9 unless another ratio is given. @mixin apply-aspect-ratio($width, $height) { padding-top: 100% * ($height/$width); } -/* Not including a default AR in vjs-fluid because it would override - the user set AR injected into the header. */ +// Not including a default AR in vjs-fluid because it would override +// the user set AR injected into the header. .video-js.vjs-fluid, .video-js.vjs-16-9, .video-js.vjs-4-3 { @@ -84,8 +83,8 @@ height: 100%; } -/* Playback technology elements expand to the width/height of the containing div -