- Sponsor
-
Notifications
You must be signed in to change notification settings - Fork 3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Bug]: medusa/ui breaks with tailwindcss v4 #11040
Comments
Removing the import allows future compatibilty with tailwind v4 where it won't be exported anymore. Fix medusajs#11040
Hey @pnodet Usually we don't make any changes to support beta versions of our dependencies, i.e. they might still change and introduce a I'll come back to you as soon as we have decided on how to proceed with this. |
Hey @christiananese I fully understand your position, we applied the PR changes via patch-package on our project so:
Since this change is quite trivial I figured it would be mergeable as is and I also don't think the |
This doesn't look beta anymore! https://www.npmjs.com/package/tailwindcss?activeTab=versions |
Nice! Thank you @pnodet. 💪 |
Removing the import allows future compatibilty with tailwind v4 where it won't be exported anymore. Fix #11040
Removing the import allows future compatibilty with tailwind v4 where it won't be exported anymore. Fix medusajs#11040
Removing the import allows future compatibilty with tailwind v4 where it won't be exported anymore. Fix medusajs#11040
Removing the import allows future compatibilty with tailwind v4 where it won't be exported anymore. Fix medusajs#11040
env
Node.js version
v22.13.0
What happended?
tailwindcss v4 is currently in beta and contains some changes. Apart from some behavior differences medusa/ui could work with it if configured correctly.
However among the changes made the tailwindcss package does not include the
defaultTheme
export which breaks since medusa/ui imports it.The text was updated successfully, but these errors were encountered: