Skip to content

Conversation

@syazwanz
Copy link

@syazwanz syazwanz commented Feb 6, 2026

Summary

The Separator component in the base and radix registries uses data-horizontal: / data-vertical: custom variant shorthands for orientation styling.

These require @import "shadcn/tailwind.css" which defines the corresponding @custom-variant.

Without it, Tailwind v4 treats them as boolean attribute checks, the styles are never applied, and the separator renders invisible (0px height/width).

This replaces the shorthands with data-[orientation=horizontal]: / data-[orientation=vertical]: selectors to match the new-york-v4 style.

Related

@vercel
Copy link
Contributor

vercel bot commented Feb 6, 2026

@syazwanz is attempting to deploy a commit to the shadcn-pro Team on Vercel.

A member of the Team first needs to authorize it.

@syazwanz syazwanz marked this pull request as draft February 6, 2026 09:30
@syazwanz syazwanz marked this pull request as ready for review February 6, 2026 09:30
@shadcn
Copy link
Collaborator

shadcn commented Feb 6, 2026

I see. any reason why you don't want the shadcn/tailwind.css custom variants?

@syazwanz
Copy link
Author

syazwanz commented Feb 7, 2026

I see. any reason why you don't want the shadcn/tailwind.css custom variants?

No issues with the custom variants themselves, but when I set up my project, @import "shadcn/tailwind.css" wasn't in my globals.css, so the separator was completely invisible. Took a while to figure out why.

Since new-york-v4 already uses the bracket syntax, this just aligns base and radix to match. Happy to close if you'd rather keep the custom variants though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants