Existing size option not applied when using Loading Button

[deogk]

<Comp
  className={cn(buttonVariants({ variant, size, className }))}
  disabled={loading}
  ref={ref}
  {...props}
>
  <>
    {loading && <Loader2 className={cn('h-4 w-4 animate-spin', children && 'mr-2')} />}
    {children}
  </>
</Comp>

add a Slottable component link: https://www.radix-ui.com/primitives/docs/utilities/slot

<Comp
  className={cn(buttonVariants({ variant, size, className }))}
  disabled={loading}
  ref={ref}
  {...props}
>
  {loading ? (
    <Slottable>
      <Loader2 className={cn('h-4 w-4 animate-spin', children && 'mr-2')} />
      {children}
    </Slottable>
  ) : (
    <Slottable>{children}</Slottable>
  )}
</Comp>

[hsuanyi-chou]

There's a slot inside the loading-button. Size also works.

[deogk]

@hsuanyi-chou Does size work with variant="link"?

[hsuanyi-chou]

yes. You can add a border in className to see the result.