From d47f2ab4fd99a69d30f043f8bbc947ec0efac135 Mon Sep 17 00:00:00 2001
From: Julian Sommer <115516918+juliansommer@users.noreply.github.com>
Date: Tue, 24 Mar 2026 14:02:26 +1000
Subject: [PATCH] fix: a11y

---
 .../src/routes/_auth/ec2/(volumes)/describe-volumes/$id.tsx | 6 ++----
 .../src/routes/_auth/s3/-components/object-list-item.tsx    | 6 ++++--
 2 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/spinifex/services/spinifexui/frontend/src/routes/_auth/ec2/(volumes)/describe-volumes/$id.tsx b/spinifex/services/spinifexui/frontend/src/routes/_auth/ec2/(volumes)/describe-volumes/$id.tsx
index 56d681825..01da69789 100644
--- a/spinifex/services/spinifexui/frontend/src/routes/_auth/ec2/(volumes)/describe-volumes/$id.tsx
+++ b/spinifex/services/spinifexui/frontend/src/routes/_auth/ec2/(volumes)/describe-volumes/$id.tsx
@@ -30,7 +30,7 @@ import {
   SelectTrigger,
   SelectValue,
 } from "@/components/ui/select"
-import { formatDateTime } from "@/lib/utils"
+import { formatDateTime, getNameTag } from "@/lib/utils"
 import {
   useAttachVolume,
   useDeleteVolume,
@@ -319,9 +319,7 @@ function VolumeDetail() {
               </SelectTrigger>
               <SelectContent>
                 {runningInstances.map((instance) => {
-                  const name = instance.Tags?.find(
-                    (t) => t.Key === "Name",
-                  )?.Value
+                  const name = getNameTag(instance.Tags)
                   return (
                     <SelectItem
                       key={instance.InstanceId}
diff --git a/spinifex/services/spinifexui/frontend/src/routes/_auth/s3/-components/object-list-item.tsx b/spinifex/services/spinifexui/frontend/src/routes/_auth/s3/-components/object-list-item.tsx
index d04c82fab..2636997c3 100644
--- a/spinifex/services/spinifexui/frontend/src/routes/_auth/s3/-components/object-list-item.tsx
+++ b/spinifex/services/spinifexui/frontend/src/routes/_auth/s3/-components/object-list-item.tsx
@@ -47,8 +47,8 @@ export function ObjectListItem({
         link.remove()
         URL.revokeObjectURL(url)
       }
-    } catch {
-      throw new Error("Failed to download object")
+    } catch (error) {
+      throw new Error("Failed to download object", { cause: error })
     }
   }
 
@@ -88,6 +88,7 @@ export function ObjectListItem({
             {formatSize(object.Size ?? 0)}
           </div>
           <button
+            aria-label={`Download ${displayName}`}
             className="rounded-md p-2 transition-colors hover:bg-accent"
             onClick={downloadObject}
             type="button"
@@ -95,6 +96,7 @@ export function ObjectListItem({
             <Download className="size-4" />
           </button>
           <button
+            aria-label={`Delete ${displayName}`}
             className="rounded-md p-2 text-destructive transition-colors hover:bg-accent"
             onClick={() => setShowDeleteDialog(true)}
             type="button"