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

Improve tailwindcss.com docs for Vite config #1939

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

visrut7
Copy link

@visrut7 visrut7 commented Jan 19, 2025

Problem:

Copy link

vercel bot commented Jan 19, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
tailwindcss-com ✅ Ready (Inspect) Visit Preview 💬 Add feedback Jan 19, 2025 9:50am

@henrikbjorn
Copy link

Not needed see https://vite.dev/guide/features#postcss

It already loads postcss so it seems you didn't configure postcss correctly.

@visrut7
Copy link
Author

visrut7 commented Jan 20, 2025

I followed this guide step by step: https://tailwindcss.com/docs/guides/vite

where we also install postcss in step 2, after that tailwind was not working so had to look it up and then found a solution on StackOverflow.

@henrikbjorn
Copy link

Works for me,

Sure you didn't install postcss after running init for tailwind ? In that case I think it wouldn't create the postcss.config.js file.

@visrut7
Copy link
Author

visrut7 commented Jan 20, 2025

Yeah, I believe I have done all the steps mentioned there but not worked, this has actually happened twice in my project I guess.

I also keep one log file in each of my projects, this habit of mine helps me to keep track of records for which commands I run in the CLI. You can check here: https://github.com/visrut7/express-react-mono/blob/main/log I see it generated postcss file as well but surprisingly it didn't work.

What I will do next time when create a project is make sure with postcss if it works I will close this PR.

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.

2 participants