From 3fa5a870755cecfc248a0c571fd01d1a3fa7169c Mon Sep 17 00:00:00 2001
From: NikolaBorislavovHristov <hristovnikola@protonmail.com>
Date: Sun, 18 Nov 2018 13:35:16 +0200
Subject: [PATCH] player styles refactored

---
 src/routes/Player/ControlBar/styles.less | 70 +++++++++++++++---------
 src/routes/Player/Player.js              |  4 +-
 src/routes/Player/styles.less            | 10 ++--
 3 files changed, 50 insertions(+), 34 deletions(-)

diff --git a/src/routes/Player/ControlBar/styles.less b/src/routes/Player/ControlBar/styles.less
index f6fe10f41..d2751115d 100644
--- a/src/routes/Player/ControlBar/styles.less
+++ b/src/routes/Player/ControlBar/styles.less
@@ -6,17 +6,9 @@
 
 .control-bar-container {
     height: @control-bar-height;
-    display: flex;
-    flex-direction: column;
-    justify-content: space-around;
-    top: initial !important;
-    padding-left: 2%;
-    padding-right: 2%;
 
     .time-slider {
-        width: 100%;
         height: @seek-slider-height;
-        z-index: 1;
 
         .time-thumb {
             width: @seek-slider-height;
@@ -25,16 +17,54 @@
     }
 
     .buttons-bar-container {
-        width: 100%;
         height: @buttons-bar-height;
-        display: flex;
-        flex-direction: row;
-        align-items: center;
-        z-index: 2;
 
         .button {
             height: @buttons-bar-height;
             width: @buttons-bar-height;
+        }
+
+        .separator {
+            margin-left: floor((@buttons-bar-height * 0.25));
+            margin-right: floor((@buttons-bar-height * 0.25));
+        }
+    }
+}
+
+.volume-slider-container {
+    padding: ceil((@buttons-bar-height * 0.4)) 0;
+    width: @buttons-bar-height;
+
+    .volume-slider {
+        width: floor((@buttons-bar-height * 0.4));
+        height: floor((@buttons-bar-height * 3));
+
+        .volume-thumb {
+            width: floor((@buttons-bar-height * 0.4));
+            height: floor((@buttons-bar-height * 0.4));
+        }
+    }
+}
+
+.control-bar-container {
+    display: flex;
+    flex-direction: column;
+    justify-content: flex-end;
+    top: initial !important;
+    padding-left: 2%;
+    padding-right: 2%;
+
+    .time-slider {
+        width: 100%;
+    }
+
+    .buttons-bar-container {
+        width: 100%;
+        display: flex;
+        flex-direction: row;
+        align-items: center;
+
+        .button {
             display: flex;
             justify-content: center;
             align-items: center;
@@ -62,8 +92,6 @@
         }
 
         .separator {
-            margin-left: floor((@buttons-bar-height * 0.25));
-            margin-right: floor((@buttons-bar-height * 0.25));
             width: 1px;
             height: 70%;
             background-color: @colorneutrallight80;
@@ -81,19 +109,7 @@
 }
 
 .volume-slider-container {
-    padding: ceil((@buttons-bar-height * 0.4)) 0;
-    width: @buttons-bar-height;
     display: flex;
     justify-content: center;
     background-color: @colorbgmain;
-
-    .volume-slider {
-        width: floor((@buttons-bar-height * 0.4));
-        height: floor((@buttons-bar-height * 3));
-
-        .volume-thumb {
-            width: floor((@buttons-bar-height * 0.4));
-            height: floor((@buttons-bar-height * 0.4));
-        }
-    }
 }
\ No newline at end of file
diff --git a/src/routes/Player/Player.js b/src/routes/Player/Player.js
index a12ada2f7..0f59c3874 100644
--- a/src/routes/Player/Player.js
+++ b/src/routes/Player/Player.js
@@ -104,7 +104,7 @@ class Player extends Component {
             <Fragment>
                 <this.state.videoComponent
                     ref={this.videoRef}
-                    className={classnames(styles['layer'], styles['video'])}
+                    className={styles['layer']}
                     onEnded={this.onEnded}
                     onError={this.onError}
                     onPropValue={this.onPropValue}
@@ -138,7 +138,7 @@ class Player extends Component {
 
     render() {
         return (
-            <div className={styles['root-container']}>
+            <div className={styles['player-container']}>
                 {this.renderVideo()}
                 {this.renderControlBar()}
             </div>
diff --git a/src/routes/Player/styles.less b/src/routes/Player/styles.less
index 5646c925f..aebb016ae 100644
--- a/src/routes/Player/styles.less
+++ b/src/routes/Player/styles.less
@@ -1,4 +1,4 @@
-.root-container {
+.player-container {
     position: relative;
     width: 100%;
     height: 100%;
@@ -10,10 +10,10 @@
         left: 0;
         right: 0;
         bottom: 0;
+    }
 
-        &.video {
-            width: 100%;
-            height: 100%;
-        }
+    video.layer {
+        width: 100%;
+        height: 100%;
     }
 }
\ No newline at end of file