From c1eba0ec3b797e94c7d2df249f9943ee8ce60c41 Mon Sep 17 00:00:00 2001
From: devgioele <gioele.devitti@aboutbits.it>
Date: Wed, 9 Aug 2023 12:12:24 +0000
Subject: [PATCH] Fix ButtonLink (#263)

* fix styles of ButtonLink

* refactor button styles
---
 src/components/button/Button.tsx         | 6 +++---
 src/components/button/ButtonIcon.tsx     | 4 +---
 src/components/button/ButtonIconLink.tsx | 4 +---
 src/components/button/ButtonLink.tsx     | 5 ++---
 src/components/button/theme.ts           | 7 ++++++-
 5 files changed, 13 insertions(+), 13 deletions(-)

diff --git a/src/components/button/Button.tsx b/src/components/button/Button.tsx
index 0fd758a9..a95affa1 100644
--- a/src/components/button/Button.tsx
+++ b/src/components/button/Button.tsx
@@ -32,9 +32,9 @@ export const Button = forwardRef<HTMLButtonElement, ButtonProps>(
       <button
         className={classNames(
           button.button.base,
-          !props.disabled
-            ? button.modeVariantTone[mode][variant][tone]
-            : button.modeVariantTone[mode][variant].disabled,
+          props.disabled
+            ? button.modeVariantTone[mode][variant].disabled
+            : button.modeVariantTone[mode][variant][tone],
           button.button.variantSize.base,
           button.button.variantSize[variant][size],
           className,
diff --git a/src/components/button/ButtonIcon.tsx b/src/components/button/ButtonIcon.tsx
index f1e920df..5aa4afff 100644
--- a/src/components/button/ButtonIcon.tsx
+++ b/src/components/button/ButtonIcon.tsx
@@ -34,9 +34,7 @@ export const ButtonIcon = forwardRef<HTMLButtonElement, ButtonIconProps>(
           !props.disabled
             ? button.modeVariantTone[mode][variant][tone]
             : button.modeVariantTone[mode][variant].disabled,
-          button.buttonIcon.variantSize[
-            variant === ButtonVariant.Ghost ? variant : 'base'
-          ][size],
+          button.buttonIcon.variantSize[variant][size],
           className,
         )}
         aria-label={label}
diff --git a/src/components/button/ButtonIconLink.tsx b/src/components/button/ButtonIconLink.tsx
index 53a70579..f93311bb 100644
--- a/src/components/button/ButtonIconLink.tsx
+++ b/src/components/button/ButtonIconLink.tsx
@@ -41,9 +41,7 @@ export const ButtonIconLink = forwardRef<
     !disabled
       ? button.modeVariantTone[mode][variant][tone]
       : button.modeVariantTone[mode][variant].disabled,
-    button.buttonIcon.variantSize[
-      variant === ButtonVariant.Ghost ? variant : 'base'
-    ][size],
+    button.buttonIcon.variantSize[variant][size],
     className,
   )
 
diff --git a/src/components/button/ButtonLink.tsx b/src/components/button/ButtonLink.tsx
index 63f0d1d4..af4a9811 100644
--- a/src/components/button/ButtonLink.tsx
+++ b/src/components/button/ButtonLink.tsx
@@ -40,9 +40,8 @@ export const ButtonLink = forwardRef<HTMLAnchorElement, ButtonLinkProps>(
       disabled
         ? button.modeVariantTone[mode][variant].disabled
         : button.modeVariantTone[mode][variant][tone],
-      button.button.variantSize[
-        variant === ButtonVariant.Ghost ? variant : 'base'
-      ][size],
+      button.button.variantSize.base,
+      button.button.variantSize[variant][size],
       className,
     )
 
diff --git a/src/components/button/theme.ts b/src/components/button/theme.ts
index 351bd50d..a09a9402 100644
--- a/src/components/button/theme.ts
+++ b/src/components/button/theme.ts
@@ -60,7 +60,7 @@ export default {
       },
     },
     variantSize: {
-      base: {
+      [ButtonVariant.Solid]: {
         [Size.Sm]: 'p-1.5',
         [Size.Md]: 'p-2',
         [Size.Lg]: 'p-3',
@@ -70,6 +70,11 @@ export default {
         [Size.Md]: 'p-2-1px border',
         [Size.Lg]: 'p-3-1px border',
       },
+      [ButtonVariant.Transparent]: {
+        [Size.Sm]: 'p-1.5',
+        [Size.Md]: 'p-2',
+        [Size.Lg]: 'p-3',
+      },
     },
   },
   modeVariantTone: {