ui icon indicating copy to clipboard operation
ui copied to clipboard

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

Open 0xdevalias opened this issue 1 year ago • 1 comments

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:

  • https://tailwindcss.com/docs/transition-property

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

  • [X] I've made research efforts and searched the documentation
  • [X] I've searched for existing issues

0xdevalias avatar Nov 13 '24 23:11 0xdevalias

This issue has been automatically marked as stale due to one year of inactivity. It will be closed in 7 days unless there’s further input. If you believe this issue is still relevant, please leave a comment or provide updated details. Thank you. (This is an automated message)

shadcn avatar Nov 13 '25 23:11 shadcn

This issue has been automatically closed due to one year of inactivity. If you’re still experiencing a similar problem or have additional details to share, please open a new issue following our current issue template. Your updated report helps us investigate and address concerns more efficiently. Thank you for your understanding! (This is an automated message)

shadcn avatar Nov 20 '25 23:11 shadcn