tailwindlabs / tailwindcss

A utility-first CSS framework for rapid UI development.
https://tailwindcss.com/
MIT License
83.4k stars 4.22k forks source link

[v4] --radius-full is not available as a variable #15115

Open BeetrootShoulders opened 3 days ago

BeetrootShoulders commented 3 days ago

What version of Tailwind CSS are you using?

4 beta.2

What build tool (or framework if it abstracts the build tool) are you using?

Tailwind Play

What version of Node.js are you using?

N/A

What browser are you using?

Firefox

What operating system are you using?

macOS

Reproduction URL

https://play.tailwindcss.com/bpvuR7VY5Z

Describe your issue

--radius-full does not appear to be available as a variable. As an obviously pointless examplerounded-(--radius-lg) will result in an element with rounded corners, rounded-(--radius-full) will not.

adamwathan commented 2 days ago

Hey! This is by design in v4 as "full" only really has one meaning so we've hard-coded values like that into core instead of making them driven by variables (it wouldn't make sense to redefine --radius-full to mean something else).

But maybe in this case it's still useful to have in there since it's a somewhat complex value to type out by hand (calc(infinity * 1px)), will leave this open until I have a chance to think about it!

philipp-spiess commented 1 day ago

@BeetrootShoulders Out of curiosity: Do you mind sharing the more specific case you were running into that lead you to notice this variable not being defined? 🙂

tobimori commented 1 day ago

I've noticed this as well, since I've tried to avoid @apply more and more and result to theme() or now var(). I agree there's no need in theory, but it's just consistency when writing CSS without utilities.

Related: https://discord.com/channels/486935104384532500/486935104384532502/1299322215094816870

BeetrootShoulders commented 1 day ago

@BeetrootShoulders Out of curiosity: Do you mind sharing the more specific case you were running into that lead you to notice this variable not being defined? 🙂

I actually opened the issue on behalf of a user on the TW Discord (where I try to help out as best I can): https://discord.com/channels/486935104384532500/486935104384532502/1309559484255047721