tailwind icon indicating copy to clipboard operation
tailwind copied to clipboard

Comment out @import directives

Open oskarkook opened this issue 2 years ago • 1 comments

Deleting the @import "./phoenix.css"; rule is not clear if you have not used Tailwind CLI before. Since installing this package is backwards-incompatible in this sense, I think it would be clearer to just comment out those lines, especially since the application will now not be able to use the classes in ./phoenix.css. The user will have a clearer understanding of what is going on, even later on, if they do not overlook the changes and commit them straight into the repo.

In comparison, the change to assets/js/app.js keeps the original behaviour for the application and is therefore not a "destructive" action.

oskarkook avatar Feb 01 '22 00:02 oskarkook

TailwindCSS CLI 3.1 now has support for the @import directive: https://tailwindcss.com/blog/tailwindcss-v3-1#built-in-support-for-css-imports-in-the-cli

philipgiuliani avatar Jun 14 '22 11:06 philipgiuliani