fix(button): move hover to enabled status to remove hover on disabled…

… buttons i don't believe hover:none is a thing
agrc · Nov 14, 2024 · 291308b · 291308b
1 parent 05857d1
commit 291308b
Showing 1 changed file with 5 additions and 5 deletions.
diff --git a/packages/utah-design-system/src/components/Button.tsx b/packages/utah-design-system/src/components/Button.tsx
@@ -17,13 +17,13 @@ let button = tv({
   variants: {
     variant: {
       primary:
-        'rounded-full bg-primary-900 text-white hover:bg-primary-700 pressed:bg-primary-800',
+        'rounded-full bg-primary-900 text-white hover:enabled:bg-primary-700 pressed:bg-primary-800',
       secondary:
-        'rounded-full bg-secondary-800 text-white hover:bg-secondary-600 pressed:bg-secondary-700',
+        'rounded-full bg-secondary-800 text-white hover:enabled:bg-secondary-600 pressed:bg-secondary-700',
       accent:
-        'rounded-full bg-accent-500 text-zinc-900 hover:bg-accent-300 pressed:bg-accent-400',
+        'rounded-full bg-accent-500 text-zinc-900 hover:enabled:bg-accent-300 pressed:bg-accent-400',
       destructive:
-        'rounded-full bg-rose-700 text-white hover:bg-rose-800 pressed:bg-rose-900',
+        'rounded-full bg-rose-700 text-white hover:enabled:bg-rose-800 pressed:bg-rose-900',
       icon: 'flex items-center justify-center border-0 p-1 text-zinc-900 hover:bg-black/5 pressed:bg-black/10 disabled:bg-transparent dark:text-zinc-300 dark:hover:bg-white/10 dark:pressed:bg-white/20',
     },
     size: {
@@ -34,7 +34,7 @@ let button = tv({
       extraLarge: 'min-h-[3.125rem] px-10 text-xl',
     },
     isDisabled: {
-      true: 'hover:none border-black/20 bg-zinc-100 text-zinc-500 dark:border-white/20 dark:bg-zinc-500 dark:text-zinc-100',
+      true: 'border-black/20 bg-zinc-100 text-zinc-500 dark:border-white/20 dark:bg-zinc-500 dark:text-zinc-100',
     },
   },
   defaultVariants: {