From df676026024a85f7485e27f4f2d1070c56a68b96 Mon Sep 17 00:00:00 2001
From: "[dyad]" <git@dyad.sh>
Date: Tue, 20 Jan 2026 14:28:11 +0100
Subject: [PATCH] [dyad] Replaced sort dropdown with icon button - wrote 1
 file(s)

---
 src/components/image-alt-display.tsx | 66 ++++++++++++++++------------
 1 file changed, 39 insertions(+), 27 deletions(-)

diff --git a/src/components/image-alt-display.tsx b/src/components/image-alt-display.tsx
index 0b166a8..440fd2f 100644
--- a/src/components/image-alt-display.tsx
+++ b/src/components/image-alt-display.tsx
@@ -5,15 +5,15 @@ import type { ImageAltData } from "@/app/actions";
 import { Checkbox } from "@/components/ui/checkbox";
 import { Label } from "@/components/ui/label";
 import { Badge } from "@/components/ui/badge";
-import { ImageOff } from "lucide-react";
+import { ImageOff, ArrowDown, ArrowUp, ArrowDownUp } from "lucide-react";
 import { formatBytes } from "@/lib/utils";
+import { Button } from "@/components/ui/button";
 import {
-  Select,
-  SelectContent,
-  SelectItem,
-  SelectTrigger,
-  SelectValue,
-} from "@/components/ui/select";
+  Tooltip,
+  TooltipContent,
+  TooltipProvider,
+  TooltipTrigger,
+} from "@/components/ui/tooltip";
 
 interface ImageAltDisplayProps {
   images: ImageAltData[];
@@ -62,6 +62,18 @@ export function ImageAltDisplay({ images }: ImageAltDisplayProps) {
     return "outline";
   };
 
+  const handleSortChange = () => {
+    if (sortOrder === "default") setSortOrder("desc");
+    else if (sortOrder === "desc") setSortOrder("asc");
+    else setSortOrder("default");
+  };
+
+  const sortTooltipText = {
+    default: "Sort by size (largest first)",
+    desc: "Sort by size (smallest first)",
+    asc: "Reset to default order",
+  };
+
   return (
     <div className="space-y-4">
       <div className="flex flex-col sm:flex-row sm:items-center sm:justify-between gap-4 p-4 bg-muted/50 rounded-lg">
@@ -87,26 +99,26 @@ export function ImageAltDisplay({ images }: ImageAltDisplayProps) {
             <Label htmlFor="missing-alt">Show only missing alt text</Label>
           </div>
         </div>
-        <div className="flex items-center gap-2">
-          <Label htmlFor="sort-order" className="text-sm flex-shrink-0">
-            Sort by:
-          </Label>
-          <Select
-            value={sortOrder}
-            onValueChange={(value) =>
-              setSortOrder(value as "default" | "asc" | "desc")
-            }
-          >
-            <SelectTrigger className="w-full sm:w-[240px]" id="sort-order">
-              <SelectValue placeholder="Select sort order" />
-            </SelectTrigger>
-            <SelectContent>
-              <SelectItem value="default">Default Order</SelectItem>
-              <SelectItem value="desc">Size (Largest to Smallest)</SelectItem>
-              <SelectItem value="asc">Size (Smallest to Largest)</SelectItem>
-            </SelectContent>
-          </Select>
-        </div>
+        <TooltipProvider>
+          <Tooltip>
+            <TooltipTrigger asChild>
+              <Button
+                variant="outline"
+                size="icon"
+                onClick={handleSortChange}
+                className="flex-shrink-0"
+              >
+                {sortOrder === "desc" && <ArrowDown className="h-4 w-4" />}
+                {sortOrder === "asc" && <ArrowUp className="h-4 w-4" />}
+                {sortOrder === "default" && <ArrowDownUp className="h-4 w-4" />}
+                <span className="sr-only">Sort by size</span>
+              </Button>
+            </TooltipTrigger>
+            <TooltipContent>
+              <p>{sortTooltipText[sortOrder]}</p>
+            </TooltipContent>
+          </Tooltip>
+        </TooltipProvider>
       </div>
       <div className="space-y-4">
         {sortedAndFilteredImages.map((image, index) => (