From 5d66970fe91118f1d21fae564d32fee6331d27c8 Mon Sep 17 00:00:00 2001
From: Jamie Pine <ijamespine@me.com>
Date: Tue, 3 Feb 2026 06:25:27 -0800
Subject: [PATCH] Refactor FloatingGenerateBox and HistoryTable components for
 improved UI consistency

- Removed unnecessary class from the FloatingGenerateBox button for cleaner styling.
- Updated HistoryTable to remove redundant class from the delete button.
- Reorganized imports in dropdown-menu for better readability.
- Enhanced useAutoUpdater hook with improved dependency management and updated icon usage.
- Deleted obsolete useAutoUpdater.ts file to streamline codebase.
---
 .../Generation/FloatingGenerateBox.tsx        |   2 +-
 app/src/components/History/HistoryTable.tsx   |   1 -
 app/src/components/ui/dropdown-menu.tsx       |  12 ++--
 app/src/hooks/useAutoUpdater.ts               |  52 ------------------
 app/src/hooks/useAutoUpdater.tsx              |   6 +-
 tauri/src-tauri/gen/Assets.car                | Bin 3847048 -> 3847048 bytes
 6 files changed, 11 insertions(+), 62 deletions(-)
 delete mode 100644 app/src/hooks/useAutoUpdater.ts

diff --git a/app/src/components/Generation/FloatingGenerateBox.tsx b/app/src/components/Generation/FloatingGenerateBox.tsx
index 2e99d337..d8d493bd 100644
--- a/app/src/components/Generation/FloatingGenerateBox.tsx
+++ b/app/src/components/Generation/FloatingGenerateBox.tsx
@@ -300,7 +300,7 @@ export function FloatingGenerateBox({
                   <Button
                     type="submit"
                     disabled={isPending || !selectedProfileId}
-                    className="h-10 w-10 rounded-full bg-accent hover:bg-accent/90 hover:scale-105 text-accent-foreground shadow-lg hover:shadow-accent/50 transition-all duration-200"
+                    className="h-10 w-10 rounded-full bg-accent hover:bg-accent/90 hover:scale-105 text-accent-foreground shadow-lg transition-all duration-200"
                     size="icon"
                   >
                     {isPending ? (
diff --git a/app/src/components/History/HistoryTable.tsx b/app/src/components/History/HistoryTable.tsx
index 7a861e19..cc670e57 100644
--- a/app/src/components/History/HistoryTable.tsx
+++ b/app/src/components/History/HistoryTable.tsx
@@ -347,7 +347,6 @@ export function HistoryTable() {
                         <DropdownMenuItem
                           onClick={() => handleDeleteClick(gen.id, gen.profile_name)}
                           disabled={deleteGeneration.isPending}
-                          className="text-destructive focus:text-destructive"
                         >
                           <HugeiconsIcon icon={Delete01Icon} size={16} className="mr-2 h-4 w-4" />
                           Delete
diff --git a/app/src/components/ui/dropdown-menu.tsx b/app/src/components/ui/dropdown-menu.tsx
index bb83b772..7a878f99 100644
--- a/app/src/components/ui/dropdown-menu.tsx
+++ b/app/src/components/ui/dropdown-menu.tsx
@@ -1,7 +1,7 @@
-import * as React from 'react';
-import * as DropdownMenuPrimitive from '@radix-ui/react-dropdown-menu';
-import { HugeiconsIcon } from '@hugeicons/react';
 import { MoreHorizontalIcon } from '@hugeicons/core-free-icons';
+import { HugeiconsIcon } from '@hugeicons/react';
+import * as DropdownMenuPrimitive from '@radix-ui/react-dropdown-menu';
+import * as React from 'react';
 import { cn } from '@/lib/utils/cn';
 
 const DropdownMenu = DropdownMenuPrimitive.Root;
@@ -74,7 +74,7 @@ const DropdownMenuItem = React.forwardRef<
   <DropdownMenuPrimitive.Item
     ref={ref}
     className={cn(
-      'relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none transition-colors focus:bg-accent focus:text-accent-foreground data-[disabled]:pointer-events-none data-[disabled]:opacity-50',
+      'relative flex cursor-default select-none items-center rounded-sm px-2 py-1.5 text-sm outline-none focus:bg-accent focus:text-accent-foreground data-[disabled]:pointer-events-none data-[disabled]:opacity-50',
       inset && 'pl-8',
       className,
     )}
@@ -155,7 +155,9 @@ const DropdownMenuSeparator = React.forwardRef<
 DropdownMenuSeparator.displayName = DropdownMenuPrimitive.Separator.displayName;
 
 const DropdownMenuShortcut = ({ className, ...props }: React.HTMLAttributes<HTMLSpanElement>) => {
-  return <span className={cn('ml-auto text-xs tracking-widest opacity-60', className)} {...props} />;
+  return (
+    <span className={cn('ml-auto text-xs tracking-widest opacity-60', className)} {...props} />
+  );
 };
 DropdownMenuShortcut.displayName = 'DropdownMenuShortcut';
 
diff --git a/app/src/hooks/useAutoUpdater.ts b/app/src/hooks/useAutoUpdater.ts
deleted file mode 100644
index e99d3ac5..00000000
--- a/app/src/hooks/useAutoUpdater.ts
+++ /dev/null
@@ -1,52 +0,0 @@
-import { useCallback, useEffect, useRef, useState } from 'react';
-import { usePlatform } from '@/platform/PlatformContext';
-import type { UpdateStatus } from '@/platform/types';
-
-// Re-export UpdateStatus for backwards compatibility
-export type { UpdateStatus };
-
-export function useAutoUpdater(checkOnMount = false) {
-  const platform = usePlatform();
-  const [status, setStatus] = useState<UpdateStatus>(platform.updater.getStatus());
-  const hasCheckedRef = useRef(false);
-
-  // Subscribe to updater status changes
-  useEffect(() => {
-    const unsubscribe = platform.updater.subscribe((newStatus) => {
-      setStatus(newStatus);
-    });
-    return unsubscribe;
-    // Empty dependency array - platform is stable from context
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [platform.updater.subscribe]);
-
-  const checkForUpdates = useCallback(async () => {
-    await platform.updater.checkForUpdates();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [platform.updater.checkForUpdates]);
-
-  const downloadAndInstall = useCallback(async () => {
-    await platform.updater.downloadAndInstall();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [platform.updater.downloadAndInstall]);
-
-  const restartAndInstall = useCallback(async () => {
-    await platform.updater.restartAndInstall();
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, [platform.updater.restartAndInstall]);
-
-  useEffect(() => {
-    if (checkOnMount && platform.metadata.isTauri && !hasCheckedRef.current) {
-      hasCheckedRef.current = true;
-      checkForUpdates();
-    }
-    // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
-
-  return {
-    status,
-    checkForUpdates,
-    downloadAndInstall,
-    restartAndInstall,
-  };
-}
diff --git a/app/src/hooks/useAutoUpdater.tsx b/app/src/hooks/useAutoUpdater.tsx
index af156495..a7e1e949 100644
--- a/app/src/hooks/useAutoUpdater.tsx
+++ b/app/src/hooks/useAutoUpdater.tsx
@@ -1,5 +1,5 @@
-import { HugeiconsIcon } from '@hugeicons/react';
 import { Download01Icon, Refresh01Icon } from '@hugeicons/core-free-icons';
+import { HugeiconsIcon } from '@hugeicons/react';
 import { useCallback, useEffect, useRef, useState } from 'react';
 import { Progress } from '@/components/ui/progress';
 import { ToastAction } from '@/components/ui/toast';
@@ -74,7 +74,7 @@ export function useAutoUpdater(options: boolean | UseAutoUpdaterOptions = false)
     }
     // Empty dependency array - only run once on mount
     // eslint-disable-next-line react-hooks/exhaustive-deps
-  }, []);
+  }, [platform.metadata.isTauri, checkOnMount, checkForUpdates]);
 
   // Show toast when update is available
   useEffect(() => {
@@ -133,7 +133,7 @@ export function useAutoUpdater(options: boolean | UseAutoUpdaterOptions = false)
     toastUpdateRef.current({
       title: (
         <div className="flex items-center gap-2">
-          <Download className="h-4 w-4 animate-pulse" />
+          <HugeiconsIcon icon={Download01Icon} size={16} className="h-4 w-4 animate-pulse" />
           <span>Downloading Update</span>
         </div>
       ),
diff --git a/tauri/src-tauri/gen/Assets.car b/tauri/src-tauri/gen/Assets.car
index df9a80fcbcf1ade9897fa1cd35d81324244f4b68..a67539b65d68f107d0fafb5402c11a5502c1f08f 100644
GIT binary patch
delta 745
zcmZwFIdW7n5XSKu2E(wg20WO>>|1-(+MDEAPfPg-z5y9UNbX=JqX@WzL$c>%a0{jg
zNS*S54auOYU#NdeDlNYM7%kxIC+TvN-bo)QMVHCeSSeYpNRkO5Czn<9WW0)s)Ss1A
z_RLirZ-03I?(^1ke<|~&jNlZE;WVtl88|yEWm4BVTY3p>0<jK`Z0{l^j!ez!P?D0y
zI<L-A5=}dvvpFof$(L~_%KKEc$4+5{f+mq%No(U&i<+Z#t04}#{0n`|ll4)TYA`l%
zA<j<7wh&2zQ<S8Y)M|-Y;(tOW>GJnP%QD#*cYWY$im`YzF$lRvqJ5;^`B;ND*|f3}
z!a03#$Dup(z1<0%gY$3!F2W_a3|HVPT!ZUy18%}P+=AQiyRiXNxC@(b5AMSQcnDkY
z2p+=|cnZ(pIlO?E@M_rGeSLJ)nOZ{R=*XAgajD{{S6)+*m|f(mV^u4cVy|1PC;wu8
zu=558wqXVe7&I8L;D&=8Km6>P)M%|U#H}^5+IdPQCt~lpnLZU)%<4eh*qprLbbdGs
N5J85++1qdXzW{9a&~yL*

delta 745
zcmZwFIdT+15CzZ}2Eni+K!66Z2(d$IYROs>r&ZNe9Xf(;fJb2B%b4*94DKKkuXPM=
z!2|}bT)r4&H68KV{Fy(a7C(QD7V!OxG@gx1UInTtF|jw9vYh@D=i<9utT$w|4TjX;
zFLQRrh1#{dpFV#0x;;Hy%6us!I0s`m4{LA%F7`{A)V0Rw%1MGH>#Go}pecGxl;on;
zCZ*u@aE?t<<7&=Uzi1}k#trLgtaO!<jfuEZmPE5qwVYLErEUDr!dzpU&w@U;lZ{a$
z9J}m7AfIE0jf$aIjG|Ob4X%i9vE!gQOXPGGv@Da&aie`!x_Y6ED#R%yRLhQncV2fn
zIhWEP$T}&yu7d8*5B4W;2`<AGxC+<cI^2MpunxE2Hr#;?xC{5-G-DH{@Bp^pAv}V|
z@C3HuDeS;Acn&Y%CA@;y@TNc5e|vJ$6y-Sips;vrQK~u;Ukho+zGdxMYBwlll1lO4
zb*A&9y>~#c3o}rl!GHw@*B|Zq{&!P6j#x}0sRj?x3R8^Ql8PL>bev<T!-050Vpmt3
Q&W~pSB1nHcd;jC`59+qh?f?J)