From 42b64c11ff4053c7452b51f5d2ba9e8da6b6aa22 Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Mon, 20 Oct 2025 13:01:14 +0100
Subject: [PATCH] Refactor actions section in case view template for improved
 readability and structure

---
 atlas/templates/atlas/case_view.html | 114 +++++++++++++--------------
 sbas/tables.py                       |   1 +
 2 files changed, 58 insertions(+), 57 deletions(-)

diff --git a/atlas/templates/atlas/case_view.html b/atlas/templates/atlas/case_view.html
index 6b052ccd..0666e2c0 100755
--- a/atlas/templates/atlas/case_view.html
+++ b/atlas/templates/atlas/case_view.html
@@ -23,64 +23,64 @@
             View my <a href='{% url "atlas:case_view" %}?author={{request.user.id}}'>cases</a>.
         </details>
         <form>
-        <details id="actions-detail" class="mt-3">
-            <summary>
-                <strong>Actions</strong>
-            </summary>
-<div class="row mb-2">
-    <div class="col-auto">
-        <select id="collection-select" name="collection_id" class="form-select form-select-sm">
+            <details id="actions-detail" class="mt-3">
+                <summary>
+                    <strong>Actions</strong>
+                </summary>
+                <div class="row mb-2">
+                    <div class="col-auto">
+                        <select id="collection-select" name="collection_id" class="form-select form-select-sm">
             <!-- Dynamically load collection options via HTMX -->
-            <option value="">Loading collections...</option>
-            <option hx-trigger="every 1s[document.getElementById('actions-detail').open] once" hx-get="{% url 'atlas:collection_options' %}" hx-target="#collection-select" hx-swap="innerHTML"></option>
-        </select>
-    </div>
-    <div class="col-auto">
-        <button id="add-to-collection-btn"
-            class="btn btn-sm btn-outline-primary"
-            hx-post="{% url 'atlas:add_cases_to_collection' %}"
-            hx-include="[name='selection']:checked, #collection-select"
-            hx-target="#action-result"
-            hx-swap="innerHTML"
-            >
-            Add selected cases to collection
-        </button>
-        <button id="remove-from-collection-btn"
-            class="btn btn-sm btn-outline-danger ms-2"
-            hx-post="{% url 'atlas:remove_cases_from_collection' %}"
-            hx-include="[name='selection']:checked, #collection-select"
-            hx-target="#action-result"
-            hx-swap="innerHTML"
-            >
-            Remove selected cases from collection
-        </button>
-        <br/>
-        <button id="set-open-access-btn"
-            class="btn btn-sm btn-outline-success ms-2"
-            hx-post="{% url 'atlas:set_open_access' %}"
-            hx-include="[name='selection']:checked"
-            hx-target="#action-result"
-            hx-swap="innerHTML"
-            >
-            Set Open Access
-        </button>
-        <button id="remove-open-access-btn"
-            class="btn btn-sm btn-outline-warning ms-2"
-            hx-post="{% url 'atlas:remove_open_access' %}"
-            hx-include="[name='selection']:checked"
-            hx-target="#action-result"
-            hx-swap="innerHTML"
-            >
-            Remove Open Access
-        </button>
-        <span id="action-result"></span>
-    </div>
-</div>
-        </details>
-        {% render_table table %}
-    </form>
+                            <option value="">Loading collections...</option>
+                            <option hx-trigger="every 1s[document.getElementById('actions-detail').open] once" hx-get="{% url 'atlas:collection_options' %}" hx-target="#collection-select" hx-swap="innerHTML"></option>
+                        </select>
+                    </div>
+                    <div class="col-auto">
+                        <button id="add-to-collection-btn"
+                                class="btn btn-sm btn-outline-primary"
+                                hx-post="{% url 'atlas:add_cases_to_collection' %}"
+                                hx-include="[name='selection']:checked, #collection-select"
+                                hx-target="#action-result"
+                                hx-swap="innerHTML"
+                        >
+                            Add selected cases to collection
+                        </button>
+                        <button id="remove-from-collection-btn"
+                                class="btn btn-sm btn-outline-danger ms-2"
+                                hx-post="{% url 'atlas:remove_cases_from_collection' %}"
+                                hx-include="[name='selection']:checked, #collection-select"
+                                hx-target="#action-result"
+                                hx-swap="innerHTML"
+                        >
+                            Remove selected cases from collection
+                        </button>
+                        <br/>
+                        <button id="set-open-access-btn"
+                                class="btn btn-sm btn-outline-success ms-2"
+                                hx-post="{% url 'atlas:set_open_access' %}"
+                                hx-include="[name='selection']:checked"
+                                hx-target="#action-result"
+                                hx-swap="innerHTML"
+                        >
+                            Set Open Access
+                        </button>
+                        <button id="remove-open-access-btn"
+                                class="btn btn-sm btn-outline-warning ms-2"
+                                hx-post="{% url 'atlas:remove_open_access' %}"
+                                hx-include="[name='selection']:checked"
+                                hx-target="#action-result"
+                                hx-swap="innerHTML"
+                        >
+                            Remove Open Access
+                        </button>
+                        <span id="action-result"></span>
+                    </div>
+                </div>
+            </details>
+            {% render_table table %}
+        </form>
 
-{% include "generic/partials/page_size_form.html" %}
+        {% include "generic/partials/page_size_form.html" %}
 
-    <div id="exam-options"></div>
+        <div id="exam-options"></div>
 {% endblock %}
diff --git a/sbas/tables.py b/sbas/tables.py
index 9685b947..b142d16b 100644
--- a/sbas/tables.py
+++ b/sbas/tables.py
@@ -40,6 +40,7 @@ class QuestionTable(tables.Table):
             "author",
         )
         sequence = ("view", "stem", "answers")#, "exams")
+        attrs = {"class": "table row-selector"}
 
     def __init__(self, data=None, *args, **kwargs):
         super().__init__(