Skip to content

Conversation

david-crespo
Copy link
Collaborator

@david-crespo david-crespo commented Jul 1, 2024

Closes #2268. I also cleaned up a couple of other variants.

I wasn't able to find any spots where we were relying on the bad setup, but we should give it a bit more testing.

Copy link

vercel bot commented Jul 1, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
console ✅ Ready (Inspect) Visit Preview Jul 1, 2024 10:14pm

plugin(({ addVariant, addUtilities, variants }) => {
// imitation of the twin.macro svg: variant. svg:text-green-500 puts green
// on an SVG that's an immediate child of the element
addVariant('svg', '& > svg')
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think when we added this, you couldn't do it inline as an arbitrary value. Now you can.

Copy link
Contributor

@charliepark charliepark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Flipped through this locally, and didn't come across anything looking broken. Looks good.

@david-crespo
Copy link
Collaborator Author

Thanks, will merge after we cut v9.

@david-crespo david-crespo merged commit 81bd2ab into main Jul 11, 2024
@david-crespo david-crespo deleted the fix-disabled-variant branch July 11, 2024 18:24
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.

Fix disabled addVariant call in Tailwind config
2 participants