feat: Add margin to scorm reports and reload button and add Reload as…

… button text (#84)
overhangio · Sep 18, 2024 · dc5a52d · dc5a52d
1 parent eabeac5
commit dc5a52d
Show file tree

Hide file tree

Showing 3 changed files with 12 additions and 2 deletions.
diff --git a/openedxscorm/static/css/scormxblock.css b/openedxscorm/static/css/scormxblock.css
@@ -91,6 +91,10 @@
     display: none;
 }
 
+.scorm-reports {
+    margin-top: 2rem;
+}
+
 /* Popup window */
 .scorm-xblock:not(.can-popup) .popup-wrapper {
     display: none;
@@ -111,3 +115,9 @@
 .scorm-xblock .reports-togglable-off {
     display: none;
 }
+.report {
+    margin: 1rem 0;
+}
+.reload-report {
+    margin: 0 1rem;
+}
diff --git a/openedxscorm/static/html/scormxblock.html b/openedxscorm/static/html/scormxblock.html
@@ -55,7 +55,7 @@ <h4>Table of contents</h4>
           placeholder="Student username or email"
           class="search-students"
         />
-        <button class="reload-report reports-togglable-off" alt="reload report"></button>
+        <button class="reload-report reports-togglable-off" alt="reload report">Reload</button>
         <div class="report"></div>
       </span>
     </div>

diff --git a/openedxscorm/static/js/src/scormxblock.js b/openedxscorm/static/js/src/scormxblock.js
@@ -156,7 +156,7 @@ function ScormXBlock(runtime, element, settings) {
         }).success(function (data) {
             reportElement.html(renderjson.set_show_to_level(1)(data));
         }).fail(function () {
-            reportElement.html("no data found");
+            reportElement.html("No data found");
         }).complete(function () {
             $(element).find(".reload-report").removeClass("reports-togglable-off");
         });