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

[1.x] Simplify Tailwind installation #155

Merged
merged 3 commits into from
May 30, 2022
Merged

[1.x] Simplify Tailwind installation #155

merged 3 commits into from
May 30, 2022

Conversation

jessarcher
Copy link
Member

This PR simplifies the Tailwind installation in Breeze:

  • Removes the redundant Inertia version of app.css.
  • Uses the @tailwind directive instead of @import, per the standard installation guide.
  • Removes the unnecessary postcss-import dependency and Mix configuration.

Note that the @import/postcss-import approach is still documented in Tailwind for advanced use cases, but I think this is beyond the scope of Breeze.

Having a standard Tailwind installation also makes it easier to transition to Vite.

@taylorotwell taylorotwell merged commit 19834b6 into 1.x May 30, 2022
@taylorotwell taylorotwell deleted the tailwind-streamline branch May 30, 2022 13:18
drewmw5 pushed a commit to drewmw5/breeze that referenced this pull request Nov 11, 2022
* Remove duplicate app.css file

* Use modern Tailwind directives

* Remove redundant dependency
slimani-dev pushed a commit to slimani-dev/breeze that referenced this pull request Jan 21, 2023
* Remove duplicate app.css file

* Use modern Tailwind directives

* Remove redundant dependency
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.

3 participants