From a2bde879aa78dce127f0beeaff9647514a07e879 Mon Sep 17 00:00:00 2001
From: grafixeyehero <grafixeyehero>
Date: Mon, 16 Sep 2019 02:45:45 +0300
Subject: [PATCH 1/5] Remove Logical Operators

---
 src/components/htmlvideoplayer/plugin.js | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/components/htmlvideoplayer/plugin.js b/src/components/htmlvideoplayer/plugin.js
index d44980191a..298662f1e4 100644
--- a/src/components/htmlvideoplayer/plugin.js
+++ b/src/components/htmlvideoplayer/plugin.js
@@ -1399,7 +1399,7 @@ define(['browser', 'require', 'events', 'apphost', 'loading', 'dom', 'playbackMa
                         }
 
                         // Can't autoplay in these browsers so we need to use the full controls, at least until playback starts
-                        if (!appHost.supports('htmlvideoautoplay')) {
+                        if (appHost.supports('htmlvideoautoplay')) {
                             html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" controls="controls" webkit-playsinline playsinline>';
                         } else {
 

From 20c23dc7285b831bdf130275c1e6aa760a7282dc Mon Sep 17 00:00:00 2001
From: grafixeyehero <grafixeyehero>
Date: Sat, 21 Sep 2019 18:15:27 +0300
Subject: [PATCH 2/5] switch lording html

---
 src/components/htmlvideoplayer/plugin.js | 9 +++------
 1 file changed, 3 insertions(+), 6 deletions(-)

diff --git a/src/components/htmlvideoplayer/plugin.js b/src/components/htmlvideoplayer/plugin.js
index 298662f1e4..8b095d40e0 100644
--- a/src/components/htmlvideoplayer/plugin.js
+++ b/src/components/htmlvideoplayer/plugin.js
@@ -1398,13 +1398,10 @@ define(['browser', 'require', 'events', 'apphost', 'loading', 'dom', 'playbackMa
                             cssClass += ' htmlvideoplayer-moveupsubtitles';
                         }
 
-                        // Can't autoplay in these browsers so we need to use the full controls, at least until playback starts
-                        if (appHost.supports('htmlvideoautoplay')) {
-                            html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" controls="controls" webkit-playsinline playsinline>';
-                        } else {
-
-                            // Chrome 35 won't play with preload none
+                        if (!appHost.supports('htmlvideoautoplay')) {
                             html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" webkit-playsinline playsinline>';
+                        } else {
+                            html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" controls="controls" webkit-playsinline playsinline>';
                         }
 
                         html += '</video>';

From 350463a1e76ed12506f1cf8f8554e04584c4bbc4 Mon Sep 17 00:00:00 2001
From: grafixeyehero <grafixeyehero>
Date: Thu, 26 Sep 2019 21:14:23 +0300
Subject: [PATCH 3/5] revert back

---
 src/components/htmlvideoplayer/plugin.js | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/src/components/htmlvideoplayer/plugin.js b/src/components/htmlvideoplayer/plugin.js
index 8b095d40e0..d44980191a 100644
--- a/src/components/htmlvideoplayer/plugin.js
+++ b/src/components/htmlvideoplayer/plugin.js
@@ -1398,10 +1398,13 @@ define(['browser', 'require', 'events', 'apphost', 'loading', 'dom', 'playbackMa
                             cssClass += ' htmlvideoplayer-moveupsubtitles';
                         }
 
+                        // Can't autoplay in these browsers so we need to use the full controls, at least until playback starts
                         if (!appHost.supports('htmlvideoautoplay')) {
-                            html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" webkit-playsinline playsinline>';
-                        } else {
                             html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" controls="controls" webkit-playsinline playsinline>';
+                        } else {
+
+                            // Chrome 35 won't play with preload none
+                            html += '<video class="' + cssClass + '" preload="metadata" autoplay="autoplay" webkit-playsinline playsinline>';
                         }
 
                         html += '</video>';

From 5d6e6c5db9cd27335039cb07c3961037cb94336c Mon Sep 17 00:00:00 2001
From: grafixeyehero <grafixeyehero>
Date: Thu, 26 Sep 2019 21:20:15 +0300
Subject: [PATCH 4/5] hiding default browser controls

---
 src/components/htmlvideoplayer/style.css | 24 ++++++++++++++----------
 1 file changed, 14 insertions(+), 10 deletions(-)

diff --git a/src/components/htmlvideoplayer/style.css b/src/components/htmlvideoplayer/style.css
index 44818db9fd..6397090733 100644
--- a/src/components/htmlvideoplayer/style.css
+++ b/src/components/htmlvideoplayer/style.css
@@ -8,9 +8,9 @@
     align-items: center;
 }
 
-    .videoPlayerContainer:not(.videoPlayerContainer-withBackdrop) {
-        background: #000 !important;
-    }
+.videoPlayerContainer:not(.videoPlayerContainer-withBackdrop) {
+    background: #000 !important;
+}
 
 .videoPlayerContainer-withBackdrop {
     background-repeat: no-repeat;
@@ -24,6 +24,10 @@
     z-index: 1000;
 }
 
+video::-webkit-media-controls {
+    display:none !important;
+}
+
 .htmlvideoplayer {
     margin: 0 !important;
     padding: 0 !important;
@@ -31,12 +35,12 @@
     height: 100%;
 }
 
-    .htmlvideoplayer::cue {
-        background-color: transparent;
-        text-shadow: .14em .14em .14em rgba(0, 0, 0, 1);
-        -webkit-font-smoothing: antialiased;
-        font-family: inherit;
-    }
+.htmlvideoplayer::cue {
+    background-color: transparent;
+    text-shadow: .14em .14em .14em rgba(0, 0, 0, 1);
+    -webkit-font-smoothing: antialiased;
+    font-family: inherit;
+}
 
 .htmlvideoplayer-moveupsubtitles::-webkit-media-text-track-display {
     /*Style the text itself*/
@@ -55,7 +59,7 @@
 
 .videoSubtitlesInner {
     max-width: 70%;
-    background-color: rgba(0,0,0,.8);
+    background-color: rgba(0, 0, 0, .8);
     padding: .25em;
     margin: auto;
     display: inline-block;

From 9bb5fe0f0b1cddaf7602b7730a592ae4b5032ada Mon Sep 17 00:00:00 2001
From: dkanada <dkanada@users.noreply.github.com>
Date: Tue, 1 Oct 2019 01:03:34 +0900
Subject: [PATCH 5/5] minor lint fixes

---
 src/components/htmlvideoplayer/style.css | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/src/components/htmlvideoplayer/style.css b/src/components/htmlvideoplayer/style.css
index 6397090733..9550f2c873 100644
--- a/src/components/htmlvideoplayer/style.css
+++ b/src/components/htmlvideoplayer/style.css
@@ -25,7 +25,7 @@
 }
 
 video::-webkit-media-controls {
-    display:none !important;
+    display: none !important;
 }
 
 .htmlvideoplayer {
@@ -37,13 +37,13 @@ video::-webkit-media-controls {
 
 .htmlvideoplayer::cue {
     background-color: transparent;
-    text-shadow: .14em .14em .14em rgba(0, 0, 0, 1);
+    text-shadow: 0.14em 0.14em 0.14em rgba(0, 0, 0, 1);
     -webkit-font-smoothing: antialiased;
     font-family: inherit;
 }
 
 .htmlvideoplayer-moveupsubtitles::-webkit-media-text-track-display {
-    /*Style the text itself*/
+    /* style the text itself */
     margin-top: -2em;
 }
 
@@ -59,15 +59,15 @@ video::-webkit-media-controls {
 
 .videoSubtitlesInner {
     max-width: 70%;
-    background-color: rgba(0, 0, 0, .8);
-    padding: .25em;
+    background-color: rgba(0, 0, 0, 0.8);
+    padding: 0.25em;
     margin: auto;
     display: inline-block;
 }
 
 @keyframes htmlvideoplayer-zoomin {
     from {
-        transform: scale3d(.2, .2, .2);
+        transform: scale3d(0.2, 0.2, 0.2);
         opacity: .6;
     }