From 82ff2c2015ac75af0771bef6385b44eee4b1f265 Mon Sep 17 00:00:00 2001
From: Ilya Topilskii <ilya@pixelpoint.io>
Date: Tue, 3 Dec 2024 12:16:29 +0100
Subject: [PATCH] refactor: update naming in label variants constant

---
 packages/ui/src/components/checkbox.tsx |  2 +-
 packages/ui/src/components/input.tsx    |  2 +-
 packages/ui/src/components/label.tsx    | 14 +++++++-------
 packages/ui/src/components/select.tsx   |  2 +-
 packages/ui/src/components/textarea.tsx |  2 +-
 5 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/packages/ui/src/components/checkbox.tsx b/packages/ui/src/components/checkbox.tsx
index 4329a2d05c..ecb6573c70 100644
--- a/packages/ui/src/components/checkbox.tsx
+++ b/packages/ui/src/components/checkbox.tsx
@@ -23,7 +23,7 @@ const Checkbox = forwardRef<ElementRef<typeof CheckboxPrimitive.Root>, CheckboxP
         </CheckboxPrimitive.Indicator>
       </CheckboxPrimitive.Root>
       {label && (
-        <Label className="leading-tight" theme="foreground-1" htmlFor={props.id}>
+        <Label className="leading-tight" color="foreground-1" htmlFor={props.id}>
           {label}
         </Label>
       )}
diff --git a/packages/ui/src/components/input.tsx b/packages/ui/src/components/input.tsx
index 95f4d82beb..b301640a7a 100644
--- a/packages/ui/src/components/input.tsx
+++ b/packages/ui/src/components/input.tsx
@@ -61,7 +61,7 @@ const Input = forwardRef<HTMLInputElement, InputProps>(
     return (
       <ControlGroup>
         {label && (
-          <Label className="mb-2.5" theme={disabled ? 'foreground-9' : 'foreground-2'} optional={optional} htmlFor={id}>
+          <Label className="mb-2.5" color={disabled ? 'foreground-9' : 'foreground-2'} optional={optional} htmlFor={id}>
             {label}
           </Label>
         )}
diff --git a/packages/ui/src/components/label.tsx b/packages/ui/src/components/label.tsx
index f7aaa7f4b8..1bb52ba5b8 100644
--- a/packages/ui/src/components/label.tsx
+++ b/packages/ui/src/components/label.tsx
@@ -11,7 +11,7 @@ const labelVariants = cva('peer-disabled:cursor-not-allowed peer-disabled:opacit
     variant: {
       default: 'text-sm font-normal leading-none'
     },
-    theme: {
+    color: {
       'foreground-1': 'text-foreground-1',
       'foreground-2': 'text-foreground-2',
       'foreground-5': 'text-foreground-5',
@@ -20,15 +20,15 @@ const labelVariants = cva('peer-disabled:cursor-not-allowed peer-disabled:opacit
   },
   defaultVariants: {
     variant: 'default',
-    theme: 'foreground-1'
+    color: 'foreground-1'
   }
 })
 
 const LabelRoot = forwardRef<
   ElementRef<typeof LabelPrimitive.Root>,
-  ComponentPropsWithoutRef<typeof LabelPrimitive.Root> & VariantProps<typeof labelVariants>
->(({ className, variant, theme, ...props }, ref) => (
-  <LabelPrimitive.Root ref={ref} className={cn(labelVariants({ variant, theme }), className)} {...props} />
+  Omit<ComponentPropsWithoutRef<typeof LabelPrimitive.Root>, 'color'> & VariantProps<typeof labelVariants>
+>(({ className, variant, color, ...props }, ref) => (
+  <LabelPrimitive.Root ref={ref} className={cn(labelVariants({ variant, color }), className)} {...props} />
 ))
 LabelRoot.displayName = LabelPrimitive.Root.displayName
 
@@ -38,9 +38,9 @@ interface LabelProps extends VariantProps<typeof labelVariants>, PropsWithChildr
   className?: string
 }
 
-const Label = ({ htmlFor, optional, theme, variant, children, className }: LabelProps) => {
+const Label = ({ htmlFor, optional, color, variant, children, className }: LabelProps) => {
   return (
-    <LabelRoot htmlFor={htmlFor} variant={variant} theme={theme} className={className}>
+    <LabelRoot htmlFor={htmlFor} variant={variant} color={color} className={className}>
       {children} {optional && <span className="text-foreground-7 align-top">(optional)</span>}
     </LabelRoot>
   )
diff --git a/packages/ui/src/components/select.tsx b/packages/ui/src/components/select.tsx
index 66cc26a149..75a075a68d 100644
--- a/packages/ui/src/components/select.tsx
+++ b/packages/ui/src/components/select.tsx
@@ -35,7 +35,7 @@ const Select: FC<SelectProps> = ({
 }) => (
   <SelectPrimitive.Root {...props}>
     {label && (
-      <Label className="mb-2.5" theme={disabled ? 'foreground-9' : 'foreground-2'} htmlFor={name}>
+      <Label className="mb-2.5" color={disabled ? 'foreground-9' : 'foreground-2'} htmlFor={name}>
         {label}
       </Label>
     )}
diff --git a/packages/ui/src/components/textarea.tsx b/packages/ui/src/components/textarea.tsx
index 38e87aab7d..79e6070cac 100644
--- a/packages/ui/src/components/textarea.tsx
+++ b/packages/ui/src/components/textarea.tsx
@@ -23,7 +23,7 @@ const Textarea = forwardRef<HTMLTextAreaElement, TextareaProps>(
     return (
       <ControlGroup>
         {label && (
-          <Label className="mb-2.5" theme={disabled ? 'foreground-9' : 'foreground-2'} optional={optional} htmlFor={id}>
+          <Label className="mb-2.5" color={disabled ? 'foreground-9' : 'foreground-2'} optional={optional} htmlFor={id}>
             {label}
           </Label>
         )}