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

Feature: Added Tailwind and associated packages #11

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

creaturenex
Copy link
Contributor

Description
Added tailwindcss postcss autoprefixer postcss-cli
configure tailwind.config / postcss.config
added script to package.json to use postcss
added tailwind to index and layout for initial tailwind test

Issues
References #9

@creaturenex
Copy link
Contributor Author

Hello @nbeck415 I got tailwind to work with pug, but from a development standpoint, every time I make a change I must stop the container, run the script for postcss. Not sure how to make it so that it executes the script every time I save. I assume because the styles.css is being edited by postcss the container must always be rebuilt.

p.s. relies on the updated makefile

@creaturenex
Copy link
Contributor Author

creaturenex commented Feb 29, 2024

@nbeck415
This works

  • The makefile build step is updated to run the postcss to generate the tailwind css styling in style.css

  • Because the build step generates the style.css file, new tailwind styling will require to rerun make develop so the image containers the relevant updated styling.

    • Should add a note to readme about rerunning make develop
    • Docker Compose watch would be a good use here.

@creaturenex
Copy link
Contributor Author

creaturenex commented Mar 18, 2024

@nbeck415 Where you able to take a look at the makefile and see this would be the best place to add the custom npm tailwind css script?

If so, this PR is ready to be reviewed.

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.

1 participant