From 7eb478b1113d5e598672bcd711a9f5f3c46b97b6 Mon Sep 17 00:00:00 2001
From: Ross <ross@xkjq.uk>
Date: Sun, 16 Nov 2025 22:26:34 +0000
Subject: [PATCH] Refactor case management links: enhance layout and improve
 button accessibility in collection detail view

---
 atlas/templates/atlas/collection_detail.html | 58 ++++++++++++--------
 1 file changed, 35 insertions(+), 23 deletions(-)

diff --git a/atlas/templates/atlas/collection_detail.html b/atlas/templates/atlas/collection_detail.html
index e625fa09..fdc5d855 100644
--- a/atlas/templates/atlas/collection_detail.html
+++ b/atlas/templates/atlas/collection_detail.html
@@ -4,30 +4,42 @@
 
 {% partialdef casedetails-management-links %}
 
-    (<a href="{% url 'atlas:collection_case_displaysetup' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-display" title="Setup default display"></i></a>
-
-    {% if casedetail.default_viewerstate %}
-        <i class="bi bi-check text-success" title="This case has a default viewerstate defined"></i>
-    {% endif %}
-    )
-
-    (<a href="{% url 'atlas:collection_case_details' casedetail.collection.pk casedetail.case.pk %}"><i class="bi bi-info-square" title="Case details"></i></a>)
-
-    {% if collection.collection_type == "QUE" %}
-        (<a href='{% url "atlas:collection_case_questions" casedetail.collection.pk casedetail.case.pk %}'>
-            {% if casedetail.question_schema %}
-                <i class="bi bi-question-square text-success" title="This case has questions defined."></i>
-            {% else %}
-                <i class="bi bi-question-square text-danger" title="This case has no questions defined."></i>
-            {% endif %}
+<div class="btn-toolbar" role="toolbar" aria-label="Case management links">
+    <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' casedetail.collection.pk casedetail.case.pk %}" title="Setup default display">
+            <i class="bi bi-display"></i>
         </a>
-        )
-    {% endif %}
 
-    {% if casedetail.case.previous_case %}
-        (<a href='{% url "atlas:collection_case_priors" casedetail.collection.pk casedetail.case.pk %}'>
-            <i class="bi bi-link-45deg" title="Manage priors"></i></a>)
-    {% endif %}
+        <a class="btn btn-outline-secondary" href="{% url 'atlas:collection_case_details' casedetail.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" casedetail.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" casedetail.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>
+
 {% endpartialdef %}
 
 {% block content %}
@@ -92,7 +104,7 @@
                 {% if casedetail.case.series.count == 0 %}
                     &nbsp;<i class="bi bi-exclamation-circle text-warning" title="No series attached to this case"></i>&nbsp;
                 {% endif %}
-                {{casedetail.case.title}}
+                {{casedetail.case.title}}&nbsp;
 
                 {% partial casedetails-management-links %}