Merge pull request #4 from Stanford-Online/giulio/fix-invideoquiz-js

Move & rename check for problem to display
Stanford-Online · Dec 15, 2016 · 32dc2f3 · 32dc2f3
2 parents 84a8460 + 711472f
commit 32dc2f3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/invideoquiz/public/css/invideoquiz.css b/invideoquiz/public/css/invideoquiz.css
@@ -16,12 +16,12 @@
 }
 #seq_content .vert-mod .vert.in-video-problem-wrapper .in-video-problem {
     position: absolute;
-    top: 50px;
+    top: 35px;
     padding: 25px 25px 0 25px;
     background: white;
     box-sizing: border-box;
     width: 100%;
-    height: 461px;
+    height: 467px;
     overflow-y: scroll;
     z-index: 99;
 }

diff --git a/invideoquiz/public/js/src/invideoquiz.js b/invideoquiz/public/js/src/invideoquiz.js
@@ -84,9 +84,9 @@ function InVideoQuizXBlock(runtime, element) {
             var problemToDisplayId = problemTimesMap[videoTime];
             if (problemToDisplayId && canDisplayProblem) {
               $('.wrapper-downloads, .video-controls', video).hide();
-              var hasProblemToDisplay = $(this).data('id').indexOf(problemToDisplayId) !== -1;
               $('#seq_content .vert-mod .vert').each(function () {
-                if (hasProblemToDisplay) {
+                var isProblemToDisplay = $(this).data('id').indexOf(problemToDisplayId) !== -1;
+                if (isProblemToDisplay) {
                   problemToDisplay = $('.xblock-student_view', this)
                   videoState.videoPlayer.pause();
                   problemToDisplay.show();