Skip to content
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

The bt-postcss config disables Tailwind config from working #315

Closed
jaredcwhite opened this issue May 21, 2021 · 2 comments
Closed

The bt-postcss config disables Tailwind config from working #315

jaredcwhite opened this issue May 21, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@jaredcwhite
Copy link
Member

Bridgetown Version:
0.21.0.beta3

To Reproduce:
If you run the tailwindcss bundled configuration, and then run bt-postcss, it seems like it breaks Tailwind from importing in the main CSS file.

Expected behavior:
Either we want it not to interfere with Tailwind working, or it should detect the existing presents of Tailwind and not continue with the repo changes.

@jaredcwhite jaredcwhite added the bug Something isn't working label May 21, 2021
@ayushn21
Copy link
Member

I can't ever see these two configurations being used in conjunction to be honest. Though I agree that this is a problem.

I propose that we add a warning when both these configurations are run saying "This will overwrite your postcss.config.js, do you want to continue?".

In the future we could probably look at doing something a bit more clever, but it's non-trivial as the order of the postcss plugins is very important and our users could have any number of plugins in their config file prior to running this configuration.

@jaredcwhite
Copy link
Member Author

Yeah, I know it's non-trival to sort this out…all I know is I tried setting up a new project using both and then was scratching my head wondering why the @tailwind directives in the index CSS were just getting output to the bundle as-is and Tailwind was broken. So maybe a warning would be helpful at the least.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants