From 712ecec25ddd966f60d209542305a384f5677401 Mon Sep 17 00:00:00 2001
From: Ross <ross@xkjq.uk>
Date: Sun, 16 Nov 2025 23:25:35 +0000
Subject: [PATCH] Enhance case search results: add data attributes for case
 identification and implement client-side removal of cases from search results
 after addition to collection

---
 .../atlas/partials/case_search_results.html   |  6 ++--
 .../atlas/partials/case_search_widget.html    | 29 +++++++++++++++++++
 atlas/views.py                                | 15 +++++++++-
 3 files changed, 46 insertions(+), 4 deletions(-)

diff --git a/atlas/templates/atlas/partials/case_search_results.html b/atlas/templates/atlas/partials/case_search_results.html
index cab4deba..284533a0 100644
--- a/atlas/templates/atlas/partials/case_search_results.html
+++ b/atlas/templates/atlas/partials/case_search_results.html
@@ -2,8 +2,8 @@
 {% if q %}
   {% if cases and cases.exists %}
     <div class="list-group mb-2">
-      {% for case in cases %}
-        <div class="list-group-item d-flex justify-content-between align-items-center">
+  {% for case in cases %}
+  <div class="list-group-item d-flex justify-content-between align-items-center" data-case-pk="{{ case.pk }}">
           <div class="flex-fill me-3">
             <div class="fw-semibold">{{ case.title }}</div>
             <div class="small text-muted">{% if case.description %}{{ case.description|truncatechars:100 }}{% endif %}</div>
@@ -28,7 +28,7 @@
     <div class="small text-muted mb-1">Recently created</div>
     <div class="list-group">
       {% for case in recent_cases %}
-        <div class="list-group-item d-flex justify-content-between align-items-center">
+        <div class="list-group-item d-flex justify-content-between align-items-center" data-case-pk="{{ case.pk }}">
           <div class="flex-fill me-3">
             <div class="fw-semibold">{{ case.title }}</div>
             <div class="small text-muted">{% if case.description %}{{ case.description|truncatechars:100 }}{% endif %}</div>
diff --git a/atlas/templates/atlas/partials/case_search_widget.html b/atlas/templates/atlas/partials/case_search_widget.html
index f5d47134..28e7c5a2 100644
--- a/atlas/templates/atlas/partials/case_search_widget.html
+++ b/atlas/templates/atlas/partials/case_search_widget.html
@@ -15,3 +15,32 @@
   </div>
 </div>
 
+<script>
+  (function(){
+    // Robust fallback: ensure repeated searches always work by using htmx.ajax when available.
+    try {
+      var input = document.getElementById('case-search-input');
+      if (!input) return;
+      var timer = null;
+      input.addEventListener('input', function(){
+        if (timer) clearTimeout(timer);
+        timer = setTimeout(function(){
+          try {
+            var q = encodeURIComponent(input.value || '');
+            var url = "{% url 'atlas:case_search' %}?q=" + q + "&collection={{ collection.pk }}";
+            if (window.htmx && typeof window.htmx.ajax === 'function') {
+              window.htmx.ajax('GET', url, { target: '#case-search-results', swap: 'innerHTML' });
+            } else {
+              // fallback to fetch
+              fetch(url, { credentials: 'same-origin' }).then(function(r){ return r.text(); }).then(function(html){
+                var el = document.getElementById('case-search-results');
+                if (el) el.innerHTML = html;
+              }).catch(function(e){ console.error('case search fetch error', e); });
+            }
+          } catch(e) { console.error(e); }
+        }, 400);
+      });
+    } catch(e) { console.error('case search widget init error', e); }
+  })();
+</script>
+
diff --git a/atlas/views.py b/atlas/views.py
index be365e65..48e7926c 100755
--- a/atlas/views.py
+++ b/atlas/views.py
@@ -1169,7 +1169,20 @@ def add_case_to_collection(request, collection_id):
             },
             request=request,
         )
-        return HttpResponse(rendered)
+        # Also instruct the client to remove this case from any visible search/recent lists
+        remove_script = (
+            '<script>'
+            '(function(){'
+            'try {'
+            "var pk = '%s';" % case.pk +
+            "document.querySelectorAll('#case-search-results [data-case-pk=\"' + pk + '\"]').forEach(function(el){ el.remove(); });" +
+            "document.querySelectorAll('#case-search-results input[name=\"case\"][value=\"' + pk + '\"]').forEach(function(inp){ var li = inp.closest('.list-group-item'); if (li) li.remove(); });" +
+            '} catch(e) { console.error("remove added case from search error", e); }'
+            '})();'
+            '</script>'
+        )
+
+        return HttpResponse(rendered + remove_script)
 
     # if request.method == "POST":