From 9ed8fd90b3dfccebfefa471f892a025b9f67df0d Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Mon, 8 Sep 2025 11:01:17 +0100
Subject: [PATCH] Enhance feedback display in exam scores template and update
 question data retrieval for improved user experience

---
 shorts/models.py                              |  2 +-
 shorts/templates/shorts/exam_scores_user.html | 30 +++++++++----------
 2 files changed, 16 insertions(+), 16 deletions(-)

diff --git a/shorts/models.py b/shorts/models.py
index e539e6de..4153ffb9 100644
--- a/shorts/models.py
+++ b/shorts/models.py
@@ -232,7 +232,7 @@ class Question(QuestionBase):
     def get_question_json(
         self,
         based: bool = True,
-        feedback: bool = False,
+        feedback: bool = True,
         answers: bool = True,
         history: bool = True,
         annotations: bool = False,
diff --git a/shorts/templates/shorts/exam_scores_user.html b/shorts/templates/shorts/exam_scores_user.html
index eb5fd585..79c376f3 100644
--- a/shorts/templates/shorts/exam_scores_user.html
+++ b/shorts/templates/shorts/exam_scores_user.html
@@ -1,22 +1,21 @@
 {% extends 'base.html' %}
 
 {% block content %}
-    <div class="question-display-block">
-        <div class="hide-show">HIDE/SHOW</div>
-        <div class="inner-display-block">
-            Question <span class="question-number">1</span>
-            <div id="single-dicom-viewer" class="dicom-viewer" data-images="" data-annotations=''>
-            </div>
-            <div class="history"></div>
-            <div class="answers-section">
+    <div class="question-display-block card mb-4" style="background-color: #111; color: #fff;">
+        <div class="card-header d-flex justify-content-between align-items-center" style="background-color: #222; color: #fff;">
+            <span>Question <span class="question-number">1</span></span>
+            <button type="button" class="btn btn-sm btn-outline-light hide-show">Hide/Show</button>
+        </div>
+        <div class="inner-display-block card-body" style="background-color: #111; color: #fff;">
+            <div id="single-dicom-viewer" class="dicom-viewer mb-3" data-images="" data-annotations=''></div>
+            <div class="history mb-3"></div>
+            <div class="answers-section mb-3">
                 <details>
-                    <summary>Sample Answers:</summary>
-                    <span class="answers">
-                    </span>
+                    <summary class="font-weight-bold" style="color: #fff;">Sample Answers:</summary>
+                    <span class="answers"></span>
                 </details>
             </div>
-            <div class="question-feedback">
-            </div>
+            <div class="question-feedback alert alert-info p-2 mb-0" style="background-color: #222; color: #fff; border-color: #444;"></div>
         </div>
     </div>
     <div class="shorts">
@@ -88,9 +87,10 @@
                             $(".question-display-block .history").empty().append(`History:<pre>${data.history}</pre>`);
                         }
                         if (data.feedback) {
-                            $(".question-display-block .question-feedback").empty().append(`Feedback:<pre>${data.feedback}</pre>`);
+                            console.log("Feedback", data.feedback)
+                            $(".question-feedback").empty().append(`Feedback:<pre>${data.feedback}</pre>`);
                         } else {
-                            $(".question-display-block .question-feedback").empty();
+                            $(".question-feedback").empty();
                         }
                         console.log(data)
                         if (data.answers && data.answers.length > 0) {