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

Confusion around expected behavior from a phx.new project and phoenix.css #28

Open
aegatlin opened this issue Feb 2, 2022 · 2 comments

Comments

@aegatlin
Copy link

aegatlin commented Feb 2, 2022

Hello, love this project and looking forward to its future <3

I have setup tailwind successfully using the README instructions but the default phx.new page CSS is broken. I suspect this is because the CSS from phoenix.css is missing. This makes sense given the changes being made, but I've spent a few hours now assuming I had done something wrong because the homepage CSS was not what I was expecting.

I can't tell if I have done something wrong.

(A) If I have done something wrong: I can't figure out how to keep the phoenix.css CSS around within the setup process.

(B) If I have not done something wrong: I suggest there be a note in the README telling me to expect the home page CSS to be messed up, since the phoenix.css CSS is now missing. And perhaps also recommend me to delete the phoenix.css file (or something like that).

@oskarkook
Copy link

oskarkook commented Feb 2, 2022

Coincidentally, I have a PR for this, which would leave a comment in the CSS file instead of deleting the import: https://github.com/phoenixframework/tailwind/pull/26/files#diff-1fa157207e3c87759b3d08b515a3a3b901f031919ecae8c487214d51578a7447R74-R79

Not that it's the best solution to this, but just one option :)

@aegatlin
Copy link
Author

aegatlin commented Feb 3, 2022

Glad to see I'm not crazy in failing to import phoenix.css XD. I also came across this blog post by Chris mentioning they will bring this in to phx.new eventually, so maybe they are ignoring phoenix.css because they are going to completly remove it from future generators. 🤷

https://fly.io/phoenix-files/tailwind-standalone/

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

No branches or pull requests

2 participants