search

shadcn-ui / ui

Beautifully designed components that you can copy and paste into your apps. Accessible. Customizable. Open Source.
https://ui.shadcn.com
MIT License
75.37k stars 4.72k forks source link

[bug]: `SidebarGroupLabel` `transition-[margin,opa]` should be `transition-[margin,opacity]` #5825

Open 0xdevalias opened 1 week ago

0xdevalias commented 1 week ago

Describe the bug

I'm fairly new to TailwindCSS, so I believe this is a bug, but may be wrong.

It seems that SidebarGroupLabel's className should use transition-[margin,opacity] instead of transition-[margin,opa]

default:

https://github.com/shadcn-ui/ui/blob/805ed4120a6a8ae6f6e9714cbd776e18eeba92c7/apps/www/registry/default/ui/sidebar.tsx#L431-L449

new-york:

https://github.com/shadcn-ui/ui/blob/805ed4120a6a8ae6f6e9714cbd776e18eeba92c7/apps/www/registry/new-york/ui/sidebar.tsx#L431-L449

Docs:

Affected component/components

SidebarGroupLabel

How to reproduce

  1. Look at the classNames
  2. Recognise that the shorthand opa isn't valid for a transition on opacity

Codesandbox/StackBlitz link

No response

Logs

No response

System Info

N/A

Before submitting