From dc5a52d904d9425570d0bb69d4d0f8deb1e3fb27 Mon Sep 17 00:00:00 2001
From: Zameel Hassan <43750093+zameel7@users.noreply.github.com>
Date: Wed, 18 Sep 2024 11:08:10 +0530
Subject: [PATCH] feat: Add margin to scorm reports and reload button and add
 Reload as button text (#84)

---
 openedxscorm/static/css/scormxblock.css   | 10 ++++++++++
 openedxscorm/static/html/scormxblock.html |  2 +-
 openedxscorm/static/js/src/scormxblock.js |  2 +-
 3 files changed, 12 insertions(+), 2 deletions(-)

diff --git a/openedxscorm/static/css/scormxblock.css b/openedxscorm/static/css/scormxblock.css
index 1998176..a54ca14 100644
--- 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
index b82001d..33622b1 100644
--- 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
index 3ddb530..d11f495 100644
--- 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");
         });