From 45e01a78c1929ceb7ea6f7a9c78b978707613444 Mon Sep 17 00:00:00 2001
From: Ross <xkjq@ymail.com>
Date: Mon, 9 Feb 2026 11:24:43 +0000
Subject: [PATCH] Refactor case management links and button structure for
 improved readability and consistency

---
 .../atlas/partials/case_list_item.html        | 70 +++++++++---------
 .../casedetails_management_links.html         | 72 +++++++++----------
 2 files changed, 71 insertions(+), 71 deletions(-)

diff --git a/atlas/templates/atlas/partials/case_list_item.html b/atlas/templates/atlas/partials/case_list_item.html
index b56e4b4c..01313e46 100644
--- a/atlas/templates/atlas/partials/case_list_item.html
+++ b/atlas/templates/atlas/partials/case_list_item.html
@@ -24,46 +24,46 @@
         </div>
 
 
-    <div class="btn-group btn-group-sm ms-2" role="group" aria-label="case-field-links">
+        <div class="btn-group btn-group-sm ms-2" role="group" aria-label="case-field-links">
         <!-- History button: opens modal populated from hidden content on page -->
-        {% if casedetail.case.history %}
-            <a class="btn btn-outline-secondary" href="#"
-               onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-history').innerHTML;"
-               data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View history">
-                <i class="bi bi-clock-history text-primary"></i>
-            </a>
-        {% else %}
-            <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No history">
-                <i class="bi bi-clock-history text-muted"></i>
-            </a>
-        {% endif %}
+            {% if casedetail.case.history %}
+                <a class="btn btn-outline-secondary" href="#"
+                   onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-history').innerHTML;"
+                   data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View history">
+                    <i class="bi bi-clock-history text-primary"></i>
+                </a>
+            {% else %}
+                <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No history">
+                    <i class="bi bi-clock-history text-muted"></i>
+                </a>
+            {% endif %}
 
         <!-- Discussion button -->
-        {% if casedetail.case.discussion %}
-            <a class="btn btn-outline-secondary" href="#"
-               onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-discussion').innerHTML;"
-               data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View discussion">
-                <i class="bi bi-chat-left-text text-primary"></i>
-            </a>
-        {% else %}
-            <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No discussion">
-                <i class="bi bi-chat-left-text text-muted"></i>
-            </a>
-        {% endif %}
+            {% if casedetail.case.discussion %}
+                <a class="btn btn-outline-secondary" href="#"
+                   onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-discussion').innerHTML;"
+                   data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View discussion">
+                    <i class="bi bi-chat-left-text text-primary"></i>
+                </a>
+            {% else %}
+                <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No discussion">
+                    <i class="bi bi-chat-left-text text-muted"></i>
+                </a>
+            {% endif %}
 
         <!-- Report button -->
-        {% if casedetail.case.report %}
-            <a class="btn btn-outline-secondary" href="#"
-               onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-report').innerHTML;"
-               data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View report">
-                <i class="bi bi-file-earmark-text text-primary"></i>
-            </a>
-        {% else %}
-            <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No report">
-                <i class="bi bi-file-earmark-text text-muted"></i>
-            </a>
-        {% endif %}
-    </div>
+            {% if casedetail.case.report %}
+                <a class="btn btn-outline-secondary" href="#"
+                   onclick="document.querySelector('#case-field-modal .modal-content').innerHTML = document.getElementById('case-field-{{ casedetail.case.pk }}-report').innerHTML;"
+                   data-bs-toggle="modal" data-bs-target="#case-field-modal" title="View report">
+                    <i class="bi bi-file-earmark-text text-primary"></i>
+                </a>
+            {% else %}
+                <a class="btn btn-outline-secondary disabled" aria-disabled="true" href="javascript:void(0);" title="No report">
+                    <i class="bi bi-file-earmark-text text-muted"></i>
+                </a>
+            {% endif %}
+        </div>
 
     </div>
 
diff --git a/atlas/templates/atlas/partials/casedetails_management_links.html b/atlas/templates/atlas/partials/casedetails_management_links.html
index 999974ba..0a430524 100644
--- a/atlas/templates/atlas/partials/casedetails_management_links.html
+++ b/atlas/templates/atlas/partials/casedetails_management_links.html
@@ -1,44 +1,44 @@
-    <div class="btn-group btn-group-sm me-2" role="group" aria-label="management-actions">
-        <a class="btn btn-outline-secondary" href="{% url 'atlas:collection_case_displaysetup_legacy' collection.pk casedetail.case.pk %}" title="Setup default display">
-            <i class="bi bi-display"></i>
+<div class="btn-group btn-group-sm me-2" role="group" aria-label="management-actions">
+    <a class="btn btn-outline-secondary" href="{% url 'atlas:collection_case_displaysetup_legacy' collection.pk casedetail.case.pk %}" title="Setup default display">
+        <i class="bi bi-display"></i>
+    </a>
+
+    <a class="btn btn-outline-secondary" href="{% url 'atlas:collection_case_details_legacy' collection.pk casedetail.case.pk %}" title="Case details">
+        <i class="bi bi-info-square"></i>
+    </a>
+
+    {% if collection.collection_type == "QUE" %}
+        <a class="btn btn-outline-secondary" href='{% url "atlas:collection_case_questions_legacy" collection.pk casedetail.case.pk %}' title="Manage questions">
+            {% if casedetail.question_schema %}
+                <i class="bi bi-question-square text-success"></i>
+            {% else %}
+                <i class="bi bi-question-square text-danger"></i>
+            {% endif %}
         </a>
+    {% endif %}
 
-        <a class="btn btn-outline-secondary" href="{% url 'atlas:collection_case_details_legacy' collection.pk casedetail.case.pk %}" title="Case details">
-            <i class="bi bi-info-square"></i>
+    {% if casedetail.case.previous_case %}
+        <a class="btn btn-outline-secondary" href='{% url "atlas:collection_case_priors_legacy" collection.pk casedetail.case.pk %}' title="Manage priors">
+            <i class="bi bi-link-45deg"></i>
         </a>
+    {% endif %}
+</div>
 
-        {% if collection.collection_type == "QUE" %}
-            <a class="btn btn-outline-secondary" href='{% url "atlas:collection_case_questions_legacy" collection.pk casedetail.case.pk %}' title="Manage questions">
-                {% if casedetail.question_schema %}
-                    <i class="bi bi-question-square text-success"></i>
-                {% else %}
-                    <i class="bi bi-question-square text-danger"></i>
-                {% endif %}
-            </a>
-        {% endif %}
 
-        {% if casedetail.case.previous_case %}
-            <a class="btn btn-outline-secondary" href='{% url "atlas:collection_case_priors_legacy" collection.pk casedetail.case.pk %}' title="Manage priors">
-                <i class="bi bi-link-45deg"></i>
-            </a>
-        {% endif %}
-    </div>
 
-    
-
-    <div class="btn-group btn-group-sm" role="group" aria-label="status-indicator">
-        {% if casedetail.default_viewerstate %}
-            <button type="button" class="btn btn-sm btn-outline-success" aria-disabled="true" title="This case has a default viewerstate defined">
-                <i class="bi bi-check"></i>
-            </button>
-        {% endif %}
-    </div>
+<div class="btn-group btn-group-sm" role="group" aria-label="status-indicator">
+    {% if casedetail.default_viewerstate %}
+        <button type="button" class="btn btn-sm btn-outline-success" aria-disabled="true" title="This case has a default viewerstate defined">
+            <i class="bi bi-check"></i>
+        </button>
+    {% endif %}
+</div>
 
         {# Remove button (HTMX) - only shown when the surrounding template has `can_edit` true #}
-        {% if can_edit %}
-            <form class="d-inline ms-2 m-0" hx-post="{% url 'atlas:remove_case_from_collection' casedetail.case.pk collection.pk %}" hx-target="closest li" hx-swap="outerHTML" hx-confirm="Remove this case from the collection?">
-                <button type="submit" class="btn btn-outline-danger btn-sm case-remove" title="Remove this case from the collection">
-                    <i class="bi bi-trash"></i>
-                </button>
-            </form>
-        {% endif %}
\ No newline at end of file
+{% if can_edit %}
+    <form class="d-inline ms-2 m-0" hx-post="{% url 'atlas:remove_case_from_collection' casedetail.case.pk collection.pk %}" hx-target="closest li" hx-swap="outerHTML" hx-confirm="Remove this case from the collection?">
+        <button type="submit" class="btn btn-outline-danger btn-sm case-remove" title="Remove this case from the collection">
+            <i class="bi bi-trash"></i>
+        </button>
+    </form>
+{% endif %}
\ No newline at end of file