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

[feat]: Add tailwind content step for vite guide #4975

Open
2 tasks done
imMatheus opened this issue Sep 26, 2024 · 1 comment · May be fixed by #5199
Open
2 tasks done

[feat]: Add tailwind content step for vite guide #4975

imMatheus opened this issue Sep 26, 2024 · 1 comment · May be fixed by #5199

Comments

@imMatheus
Copy link

Feature description

In the guide for setting up shadcnui with vite, https://ui.shadcn.com/docs/installation/vite, I suggest we add a new section between current 2 and 3 that tells the user to change their content field in tailwind.config.js to include content: ['./index.html', './src/**/*.{js,ts,jsx,tsx}'].

Screenshot 2024-09-26 at 18 29 02

This is a step that is needed to get it to work. I followed the steps that there currently in the docs and it did not work as I got the following issue:

No Tailwind CSS configuration found at /Users/me/Desktop/project
It is likely you do not have Tailwind CSS installed or have an invalid configuration.
Install Tailwind CSS then try again.
Visit https://tailwindcss.com/docs/guides/vite to get started.

So its a necessary step to get it to work now, after I changed the content field it started working. It's also what Tailwind has right after installing the package so makes sense to include tbh

Screenshot 2024-09-26 at 18 31 58

Affected component/components

No response

Additional Context

I could make a quick simple PR for this to get it out 🙏

Before submitting

  • I've made research efforts and searched the documentation
  • I've searched for existing issues and PRs
@iainbrux
Copy link

I agree this is a very important step missing from the documentation that is required. This caused a small headache for an hour or two at the start of the project but adding the above from the tailwind documentation fixed the issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants