From ef79ddd0de35a4e10b8b4588e67938f86d2bae10 Mon Sep 17 00:00:00 2001
From: Tom <20648924+moT01@users.noreply.github.com>
Date: Fri, 24 Apr 2020 13:50:58 -0500
Subject: [PATCH] feat: add loader on video challenges (#38633)

---
 client/src/templates/Challenges/video/Show.js | 24 ++++++++++++++++---
 .../src/templates/Challenges/video/show.css   | 21 ++++++++++++++++
 2 files changed, 42 insertions(+), 3 deletions(-)

diff --git a/client/src/templates/Challenges/video/Show.js b/client/src/templates/Challenges/video/Show.js
index d24b52ebb58..0606416cd5a 100644
--- a/client/src/templates/Challenges/video/Show.js
+++ b/client/src/templates/Challenges/video/Show.js
@@ -18,6 +18,7 @@ import ChallengeDescription from '../components/Challenge-Description';
 import Spacer from '../../../components/helpers/Spacer';
 import CompletionModal from '../components/CompletionModal';
 import Hotkeys from '../components/Hotkeys';
+import Loader from '../../../components/helpers/Loader';
 import {
   isChallengeCompletedSelector,
   challengeMounted,
@@ -72,7 +73,8 @@ export class Project extends Component {
       downloadURL: null,
       selectedOption: 0,
       answer: 1,
-      showWrong: false
+      showWrong: false,
+      videoIsLoaded: false
     };
 
     this.handleSubmit = this.handleSubmit.bind(this);
@@ -136,6 +138,12 @@ export class Project extends Component {
     });
   };
 
+  videoIsReady = () => {
+    this.setState({
+      videoIsLoaded: true
+    });
+  };
+
   render() {
     const {
       data: {
@@ -171,9 +179,19 @@ export class Project extends Component {
                 <ChallengeTitle isCompleted={isChallengeCompleted}>
                   {blockNameTitle}
                 </ChallengeTitle>
-                <div style={{ textAlign: 'center' }}>
+                <div className='video-wrapper'>
+                  {!this.state.videoIsLoaded ? (
+                    <div className='video-placeholder-loader'>
+                      <Loader />
+                    </div>
+                  ) : null}
                   <YouTube
-                    onEnd={openCompletionModal}
+                    className={
+                      this.state.videoIsLoaded
+                        ? 'display-youtube-video'
+                        : 'hide-youtube-video'
+                    }
+                    onReady={this.videoIsReady}
                     opts={{ rel: 0 }}
                     videoId={videoId}
                   />
diff --git a/client/src/templates/Challenges/video/show.css b/client/src/templates/Challenges/video/show.css
index 5879c8b9d26..eea7bda54e5 100644
--- a/client/src/templates/Challenges/video/show.css
+++ b/client/src/templates/Challenges/video/show.css
@@ -1,3 +1,24 @@
+.video-wrapper {
+  display: flex;
+  flex-direction: column;
+  justify-content: space-evenly;
+  align-items: center;
+}
+
+.video-placeholder-loader {
+  min-height: 360px;
+  display: flex;
+  align-items: center;
+}
+
+.display-youtube-video {
+  display: block;
+}
+
+.hide-youtube-video {
+  display: none;
+}
+
 .video-quiz-options {
   padding: 1px 16px;
   background-color: var(--tertiary-background);