From 16387fff770c0af39f75d47ef31920099c473ddc Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Mon, 20 Oct 2025 11:41:01 +0100
Subject: [PATCH] Refactor LLM import preview to improve layout and remove
 redundant controls

---
 .../sbas/partials/import_preview.html         | 19 +++++++------------
 1 file changed, 7 insertions(+), 12 deletions(-)

diff --git a/sbas/templates/sbas/partials/import_preview.html b/sbas/templates/sbas/partials/import_preview.html
index e0dffc24..1623e6c7 100644
--- a/sbas/templates/sbas/partials/import_preview.html
+++ b/sbas/templates/sbas/partials/import_preview.html
@@ -6,29 +6,29 @@
       <div class="card-body">
         <h5 class="card-title">Question {{ item.index|add:1 }}</h5>
         {% if item.payload %}
-        <p><strong>Title:</strong> {{ item.payload.title|default:'(No title)'|safe }}</p>
+          <p><strong>Title:</strong> {{ item.payload.title|default:'(No title)'|safe }}</p>
           <p><strong>Stem:</strong> {{ item.payload.stem|default:''|safe }}</p>
           <p><strong>Answers:</strong>
             <ul>
               <li>A: {{ item.payload.a_answer|default:''|safe }}</li>
               {% if item.payload.a_feedback %}
-              <p><strong>A Feedback:</strong> {{ item.payload.a_feedback|default:''|safe }}</p>
+                <p><strong>A Feedback:</strong> {{ item.payload.a_feedback|default:''|safe }}</p>
               {% endif %}
               <li>B: {{ item.payload.b_answer|default:''|safe }}</li>
               {% if item.payload.b_feedback %}
-              <p><strong>B Feedback:</strong> {{ item.payload.b_feedback|default:''|safe }}</p>
+                <p><strong>B Feedback:</strong> {{ item.payload.b_feedback|default:''|safe }}</p>
               {% endif %}
               <li>C: {{ item.payload.c_answer|default:''|safe }}</li>
               {% if item.payload.c_feedback %}
-              <p><strong>C Feedback:</strong> {{ item.payload.c_feedback|default:''|safe }}</p>
+                <p><strong>C Feedback:</strong> {{ item.payload.c_feedback|default:''|safe }}</p>
               {% endif %}
               <li>D: {{ item.payload.d_answer|default:''|safe }}</li>
               {% if item.payload.d_feedback %}
-              <p><strong>D Feedback:</strong> {{ item.payload.d_feedback|default:''|safe }}</p>
+                <p><strong>D Feedback:</strong> {{ item.payload.d_feedback|default:''|safe }}</p>
               {% endif %}
               <li>E: {{ item.payload.e_answer|default:''|safe }}</li>
               {% if item.payload.e_feedback %}
-              <p><strong>E Feedback:</strong> {{ item.payload.e_feedback|default:''|safe }}</p>
+                <p><strong>E Feedback:</strong> {{ item.payload.e_feedback|default:''|safe }}</p>
               {% endif %}
             </ul>
           </p>
@@ -98,9 +98,4 @@
   });
 </script>
 
-<div class="d-flex gap-2">
-  <button class="btn btn-success" hx-post="{% url 'sbas:import_llm_confirm' %}" hx-target="#import-result" hx-include="closest form">Import selected</button>
-  <button class="btn btn-secondary" hx-get="{% url 'sbas:import_llm_questions' %}" hx-target="#import-result">Cancel</button>
-</div>
-
-<div id="import-result"></div>
+<!-- Bottom controls removed: submit and cancel are inside the form above -->