refactor(ui): change Button icon prop to accept component reference instead of JSX element
This commit is contained in:
@@ -80,7 +80,7 @@ const UsersPageClient = () => {
|
|||||||
variant="secondary"
|
variant="secondary"
|
||||||
size="sm"
|
size="sm"
|
||||||
onClick={() => setEditingUserId(user.id)}
|
onClick={() => setEditingUserId(user.id)}
|
||||||
icon={<PencilEdit01Icon className="w-4 h-4" />}
|
icon={PencilEdit01Icon}
|
||||||
>
|
>
|
||||||
Modifier
|
Modifier
|
||||||
</Button>
|
</Button>
|
||||||
|
|||||||
@@ -56,6 +56,8 @@ const Button = ({
|
|||||||
}
|
}
|
||||||
};
|
};
|
||||||
|
|
||||||
|
const Icon = icon;
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<button
|
<button
|
||||||
type={type}
|
type={type}
|
||||||
@@ -68,12 +70,12 @@ const Button = ({
|
|||||||
<LoadingSpinner />
|
<LoadingSpinner />
|
||||||
) : (
|
) : (
|
||||||
<>
|
<>
|
||||||
{icon && iconPosition === 'left' && (
|
{Icon && iconPosition === 'left' && (
|
||||||
<span className={iconSizes[size]}>{icon}</span>
|
<span className={iconSizes[size]}><Icon /></span>
|
||||||
)}
|
)}
|
||||||
{children}
|
{children}
|
||||||
{icon && iconPosition === 'right' && (
|
{Icon && iconPosition === 'right' && (
|
||||||
<span className={iconSizes[size]}>{icon}</span>
|
<span className={iconSizes[size]}><Icon /></span>
|
||||||
)}
|
)}
|
||||||
</>
|
</>
|
||||||
)}
|
)}
|
||||||
|
|||||||
@@ -92,7 +92,7 @@ const Modal = ({
|
|||||||
variant="ghost"
|
variant="ghost"
|
||||||
size="sm"
|
size="sm"
|
||||||
onClick={onClose}
|
onClick={onClose}
|
||||||
icon={<Cancel01Icon className="w-4 h-4" />}
|
icon={Cancel01Icon}
|
||||||
className="!p-1 -mr-1"
|
className="!p-1 -mr-1"
|
||||||
/>
|
/>
|
||||||
)}
|
)}
|
||||||
|
|||||||
Reference in New Issue
Block a user