-
Notifications
You must be signed in to change notification settings - Fork 200
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
Tailwind nixpkgs #1839
Tailwind nixpkgs #1839
Conversation
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.
Thanks @kodeFant I was able to make it work for me 💯 . I've added a few comments for your consideration.
@@ -67,7 +79,6 @@ Create the tailwind configuration file at `tailwind/tailwind.config.js` with the | |||
const plugin = require('tailwindcss/plugin'); | |||
|
|||
module.exports = { | |||
mode: 'jit', |
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.
👍
Guide/tailwindcss.markdown
Outdated
|
||
Install Tailwind along with PostCSS and some handy libraries via NPM: | ||
Usage: |
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.
I don't thing we need to show all that
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.
Can probably just remove it and just display the command
Guide/tailwindcss.markdown
Outdated
### Installing Tailwind | ||
``` | ||
$ tailwindcss | ||
tailwindcss v3.2.7 |
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.
This is what I got
tailwindcss v3.2.7 | |
tailwindcss v3.3.3 |
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.
I am guessing you are running on master, and I am running on v1.1.0.
Maybe not necessary to have any of the output there
Co-authored-by: Amitai Burstein <[email protected]>
Co-authored-by: Amitai Burstein <[email protected]>
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.
LGTM
Thanks 🙏 |
This updates the guide to use Tailwind CSS (with official plugins like
@tailwind/forms
) directly via nixpkgsNotes:
npm
guide with the nix approach. Let me know if there still are compelling reasons to keep the npm guide as wellautoprefixer
is already bundled into the Tailwind CLI, and thepostcss
package isn't really being used for anything, so this setup should be on full feature parity with the former way