From 74c5f4d8fd68e107c3c0162583c24d297baedd3a Mon Sep 17 00:00:00 2001 From: Ross Date: Mon, 27 Oct 2025 11:19:07 +0000 Subject: [PATCH] Enhance question review template with active filters and improved metadata display --- .../sbas/question_review_question.html | 178 +++++++++++------- 1 file changed, 106 insertions(+), 72 deletions(-) diff --git a/sbas/templates/sbas/question_review_question.html b/sbas/templates/sbas/question_review_question.html index ec5e99a2..f98d1aea 100644 --- a/sbas/templates/sbas/question_review_question.html +++ b/sbas/templates/sbas/question_review_question.html @@ -1,87 +1,121 @@ {% extends 'sbas/base.html' %} {% block content %} -
-
Reviewing question: {{ question.title |default:"No Title" }}
+
+ Active filters: + {% if filters.category %} + Category: {{ filters.category }} + {% else %} + Category: Any + {% endif %} + {% if filters.status %} + Status: {{ filters.status }} + {% else %} + Status: Any + {% endif %} +
Reviewing question: {{ question.title |default:"No Title" }}
- View full question details - Edit question + View full question details + Edit question - Active filters: - {% if filters.category %} - Category: {{ filters.category }} - {% else %} - Category: Any - {% endif %} - {% if filters.status %} - Status: {{ filters.status }} - {% else %} - Status: Any - {% endif %} -
-
{{ question.stem|safe }}
-
+
+
Metadata
+
    +
  • Category: {% if question.category %}{{ question.category }}{% else %}—{% endif %}
    • +
  • Author(s): + {% if question.author.count %} + {% for a in question.author.all %} + {{ a.get_full_name|default:a.username }}{% if not forloop.last %}, {% endif %} + {% endfor %} + {% else %}—{% endif %} +
    • +
  • Created: {{ question.created_date|date:"Y-m-d H:i" }}
    • +
  • Open: {% if question.open_access %}Yes{% else %}No{% endif %}
    • + {% if question.sources %} +
  • Sources: {{ question.sources|length }} item{{ question.sources|length|pluralize }}
    • + {% endif %} + {% if question.finding.count %} +
  • Findings: {{ question.finding.count }}
    • + {% endif %} + {% if question.structure.count %} +
  • Structures: {{ question.structure.count }}
    • + {% endif %} + {% if question.condition.count %} +
  • Conditions: {{ question.condition.count }}
    • + {% endif %} + {% if question.presentation.count %} +
  • Presentations: {{ question.presentation.count }}
    • + {% endif %} + {% if question.subspecialty.count %} +
  • Subspecialties: {{ question.subspecialty.count }}
    • + {% endif %} +
+
+ +
+
Stem: {{ question.stem|safe }}
+
{# Multiple-choice answers (SBA specific) #} -
- {% if question.a_answer %} -
-
A. {{ question.a_answer|safe }}
- {% if question.best_answer == 'a' %}Correct{% endif %} - {% if question.a_feedback %} -
Feedback: {{ question.a_feedback|safe }}
- {% endif %} -
- {% endif %} - {% if question.b_answer %} -
-
B. {{ question.b_answer|safe }}
- {% if question.best_answer == 'b' %}Correct{% endif %} - {% if question.b_feedback %} -
Feedback: {{ question.b_feedback|safe }}
- {% endif %} -
- {% endif %} - {% if question.c_answer %} -
-
C. {{ question.c_answer|safe }}
- {% if question.best_answer == 'c' %}Correct{% endif %} - {% if question.c_feedback %} -
Feedback: {{ question.c_feedback|safe }}
- {% endif %} -
- {% endif %} - {% if question.d_answer %} -
-
D. {{ question.d_answer|safe }}
- {% if question.best_answer == 'd' %}Correct{% endif %} - {% if question.d_feedback %} -
Feedback: {{ question.d_feedback|safe }}
- {% endif %} -
- {% endif %} - {% if question.e_answer %} -
-
E. {{ question.e_answer|safe }}
- {% if question.best_answer == 'e' %}Correct{% endif %} - {% if question.e_feedback %} -
Feedback: {{ question.e_feedback|safe }}
- {% endif %} -
- {% endif %} -
+
+ {% if question.a_answer %} +
+
A. {{ question.a_answer|safe }}
+ {% if question.best_answer == 'a' %}Correct{% endif %} + {% if question.a_feedback %} +
Feedback: {{ question.a_feedback|safe }}
+ {% endif %} +
+ {% endif %} + {% if question.b_answer %} +
+
B. {{ question.b_answer|safe }}
+ {% if question.best_answer == 'b' %}Correct{% endif %} + {% if question.b_feedback %} +
Feedback: {{ question.b_feedback|safe }}
+ {% endif %} +
+ {% endif %} + {% if question.c_answer %} +
+
C. {{ question.c_answer|safe }}
+ {% if question.best_answer == 'c' %}Correct{% endif %} + {% if question.c_feedback %} +
Feedback: {{ question.c_feedback|safe }}
+ {% endif %} +
+ {% endif %} + {% if question.d_answer %} +
+
D. {{ question.d_answer|safe }}
+ {% if question.best_answer == 'd' %}Correct{% endif %} + {% if question.d_feedback %} +
Feedback: {{ question.d_feedback|safe }}
+ {% endif %} +
+ {% endif %} + {% if question.e_answer %} +
+
E. {{ question.e_answer|safe }}
+ {% if question.best_answer == 'e' %}Correct{% endif %} + {% if question.e_feedback %} +
Feedback: {{ question.e_feedback|safe }}
+ {% endif %} +
+ {% endif %} +
{# Generic feedback text if present #} - {% if question.feedback %} -
Feedback: {{ question.feedback|safe }}
- {% endif %} -
+ {% if question.feedback %} +
Feedback: {{ question.feedback|safe }}
+ {% endif %} +
{% comment %} Add Review {% endcomment %} -
- {% include 'generic/partials/question_review_form.html' with app_name='sbas' question=question form=form %} -
+
+ {% include 'generic/partials/question_review_form.html' with app_name='sbas' question=question form=form %} +
{% endblock %} \ No newline at end of file