Skip to content

Conversation

thecrypticace
Copy link
Contributor

Fixes #1463

We'll have to bail in some cases that we didn't before to fix this. This seems fine though. The core problem was that given this theme:

@theme {
  --radius: calc(var(--radius));
}

We'd see this CSS:

.rounded {
  border-radius: calc(var(--radius));
}

And we'd try to replace variables in calc(var(--radius)). This doesn't start with a var(…) so our recursion guards didn't catch this.

This entire system is a bit fragile and I'll be working to clean this up along with how we handle pixel equivalents, light-dark replacements, color-mix replacements, etc… in some future PRs (#1330 is one of those but I want to reorganize + tweak some code first in an earlier PR which will make that one "nicer")

Found while investigating the problem further… 🤦‍♂️
This isn’t “great” but it’s a good stop gap measure for now
@thecrypticace thecrypticace merged commit 772275d into main Oct 6, 2025
12 checks passed
@thecrypticace thecrypticace deleted the fix/issue-1463 branch October 6, 2025 14:06
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.

Tailwind server hangs when config includes border radius default
2 participants