-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
fix(@astrojs/tailwind): manually load postcss config file #5908
Conversation
🦋 Changeset detectedLatest commit: 7863f66 The changes in this PR will be included in the next version bump. Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Thanks! Can you add a changeset? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great! with a couple of nits. Thanks for handling this unintentional regression 🙏
Made the requested changes |
Changes
Fixes a bug spotted from the Astro discord: https://discord.com/channels/830184174198718474/1065463699508297758
The
@astrojs/tailwind
integration internally inlines the tailwind and autoprefixer plugins in the vite config, which causes vite to ignore loading postcss configuration files.This change aims to bring back the default behavior of vite, which is to load postcss config files only when there's no inline postcss configuration in the vite user config.
Testing
Manually tested
Docs
No docs needed, this is a bug fix