From ad4e971e774420663fd997f0a353129a87dea8b3 Mon Sep 17 00:00:00 2001
From: Markus Unterwaditzer <markus-tarpit+git@unterwaditzer.net>
Date: Sat, 2 Aug 2025 22:26:22 +0200
Subject: [PATCH] Restyle download links in table view

They're taking up too much space as individual columns.

Also change DownloadLink to be a real link, so that "Copy URL" works.
---
 daemon/web/src/lib/components/DownloadLink.svelte     | 10 +++-------
 daemon/web/src/lib/components/ManifestTable.svelte    |  4 +---
 daemon/web/src/lib/components/ManifestTableRow.svelte | 10 +++++++---
 3 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/daemon/web/src/lib/components/DownloadLink.svelte b/daemon/web/src/lib/components/DownloadLink.svelte
index f42f586..d49f239 100644
--- a/daemon/web/src/lib/components/DownloadLink.svelte
+++ b/daemon/web/src/lib/components/DownloadLink.svelte
@@ -8,20 +8,16 @@
         text: string;
         full_button?: boolean;
     } = $props();
-
-    function download() {
-        window.location.href = url;
-    }
 </script>
 
-<button
+<a
+    href={url}
     class="flex flex-row {full_button
         ? 'bg-blue-500 hover:bg-blue-700 text-white font-bold py-2 px-2 sm:px-4 rounded-md'
         : 'text-blue-600 underline'}"
-    onclick={download}
 >
     {text}
     <svg class="fill-current w-4 h-4 m-1" xmlns="http://www.w3.org/2000/svg" viewBox="0 0 20 20">
         <path d="M13 8V2H7v6H2l8 8 8-8h-5zM0 18h20v2H0v-2z" />
     </svg>
-</button>
+</a>
diff --git a/daemon/web/src/lib/components/ManifestTable.svelte b/daemon/web/src/lib/components/ManifestTable.svelte
index c07ce04..b9c47c3 100644
--- a/daemon/web/src/lib/components/ManifestTable.svelte
+++ b/daemon/web/src/lib/components/ManifestTable.svelte
@@ -19,9 +19,7 @@
             <th class="p-2" scope="col">Started</th>
             <th class="p-2" scope="col">Last Message</th>
             <th class="p-2" scope="col">Size</th>
-            <th class="p-2" scope="col">PCAP</th>
-            <th class="p-2" scope="col">QMDL</th>
-            <th class="p-2" scope="col">ZIP</th>
+            <th class="p-2" scope="col">Download</th>
             <th class="p-2" scope="col">Analysis</th>
             <th class="p-2" scope="col"></th>
         </tr>
diff --git a/daemon/web/src/lib/components/ManifestTableRow.svelte b/daemon/web/src/lib/components/ManifestTableRow.svelte
index f685d29..211a7b2 100644
--- a/daemon/web/src/lib/components/ManifestTableRow.svelte
+++ b/daemon/web/src/lib/components/ManifestTableRow.svelte
@@ -43,9 +43,13 @@
         >{(entry.last_message_time && date_formatter.format(entry.last_message_time)) || 'N/A'}</td
     >
     <td class="p-2">{entry.get_readable_qmdl_size()}</td>
-    <td class="p-2"><DownloadLink url={entry.get_pcap_url()} text="pcap" /></td>
-    <td class="p-2"><DownloadLink url={entry.get_qmdl_url()} text="qmdl" /></td>
-    <td class="p-2"><DownloadLink url={entry.get_zip_url()} text="zip" /></td>
+    <td class="p-2">
+        <div class="flex flex-row gap-2">
+            <DownloadLink url={entry.get_pcap_url()} text="pcap" />
+            <DownloadLink url={entry.get_qmdl_url()} text="qmdl" />
+            <DownloadLink url={entry.get_zip_url()} text="zip" />
+        </div>
+    </td>
     <td class="p-2"
         ><AnalysisStatus onclick={toggle_analysis_visibility} {entry} {analysis_visible} /></td
     >