From 4762ff4f571e0c12f18dc66efe0fcf0fc6e7c584 Mon Sep 17 00:00:00 2001
From: Nick Sweeting <git@sweeting.me>
Date: Fri, 28 Aug 2026 21:14:59 -0700
Subject: [PATCH] fix: soften archive result delete controls

---
 archivebox/core/admin_archiveresults.py                  | 2 +-
 archivebox/templates/core/snapshot.html                  | 2 +-
 .../templates/includes/output_delete_controls.html       | 9 +++++++--
 archivebox/tests/test_ui_admin_snapshot.py               | 6 ++++++
 4 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/archivebox/core/admin_archiveresults.py b/archivebox/core/admin_archiveresults.py
index 7aeb8736..8b4b81dd 100644
--- a/archivebox/core/admin_archiveresults.py
+++ b/archivebox/core/admin_archiveresults.py
@@ -198,7 +198,7 @@ def render_archiveresults_list(archiveresults_qs, limit=50, config=None, can_del
         if can_delete:
             delete_button = f'''
                 <button type="button" data-archive-result-ids="{result.id}" data-delete-url="{delete_url}"
-                        title="Delete this output">❌</button>
+                        title="Delete this output">×</button>
             '''
 
         rows.append(f'''
diff --git a/archivebox/templates/core/snapshot.html b/archivebox/templates/core/snapshot.html
index 8e638d28..8fb3cbb7 100644
--- a/archivebox/templates/core/snapshot.html
+++ b/archivebox/templates/core/snapshot.html
@@ -1712,7 +1712,7 @@
                                             <a href="{{display_url}}" data-no-preview="1" title="Download output file" download>⬇️</a>
                                         {% endif %}
                                         {% if can_delete_outputs and result.result %}
-                                            <button type="button" data-no-preview="1" data-archive-result-ids="{{result.result_ids}}" data-delete-url="{% admin_base_url %}/admin/core/archiveresult/" title="Delete this output">❌</button>
+                                            <button type="button" data-no-preview="1" data-archive-result-ids="{{result.result_ids}}" data-delete-url="{% admin_base_url %}/admin/core/archiveresult/" title="Delete this output">×</button>
                                         {% endif %}
                                     </div>
                                     {% if display_path %}
diff --git a/archivebox/templates/includes/output_delete_controls.html b/archivebox/templates/includes/output_delete_controls.html
index ca70c6a8..8f9c04df 100644
--- a/archivebox/templates/includes/output_delete_controls.html
+++ b/archivebox/templates/includes/output_delete_controls.html
@@ -1,3 +1,8 @@
+<style>
+    [data-archive-result-ids] { color: #64748b; }
+    [data-archive-result-ids]:hover,
+    [data-archive-result-ids].delete-pending { color: #b91c1c; }
+</style>
 <script>
     (() => {
         for (const sortButton of document.querySelectorAll('[data-output-size-sort]')) {
@@ -23,8 +28,8 @@
         function renderDeleteCountdown(label='') {
             for (const button of deleteOutputButtons) {
                 const queued = button.dataset.archiveResultIds.split(',').every((id) => queuedOutputIds.has(id))
-                button.textContent = queued ? label : '❌'
-                button.style.color = queued ? '#b91c1c' : ''
+                button.textContent = queued ? label : '×'
+                button.classList.toggle('delete-pending', queued)
             }
         }
 
diff --git a/archivebox/tests/test_ui_admin_snapshot.py b/archivebox/tests/test_ui_admin_snapshot.py
index 41ea340a..ac91a82f 100644
--- a/archivebox/tests/test_ui_admin_snapshot.py
+++ b/archivebox/tests/test_ui_admin_snapshot.py
@@ -780,6 +780,10 @@ class TestSnapshotOutputDeletion:
         assert "action: 'delete_selected'" in html
         assert "/admin/core/archiveresult/" in html
         assert "[deleting]" in html
+        assert ">×</button>" in html
+        assert "[data-archive-result-ids]:hover" in html
+        assert "[data-archive-result-ids].delete-pending" in html
+        assert "button.classList.toggle('delete-pending', queued)" in html
 
         request.user = AnonymousUser()
         anonymous_html = SnapshotView.render_live_index(request, snapshot).content.decode()
@@ -856,6 +860,8 @@ class TestSnapshotOutputDeletion:
         assert reverse("admin:core_archiveresult_changelist") in html
         assert "const queuedOutputIds = new Set()" in html
         assert "action: 'delete_selected'" in html
+        assert ">×</button>" in html
+        assert "button.classList.toggle('delete-pending', queued)" in html
 
 
 class TestAdminSnapshotListView: