build(tailwind): upgrade tailwind to v3.1.6#88
build(tailwind): upgrade tailwind to v3.1.6#88gupta-ji6 merged 8 commits intoscriptified:mainfrom hussamkhatib:main
Conversation
|
Sorry for responding so late, we missed reviewing this PR |
|
@hussamkhatib You would need to run the purge script as well with the You can use a combination of |
remove purgeCSS and use lookup table for dynamic classes.
|
@all-contributors please add @hussamkhatib for code, design, maintenance |
|
I've put up a pull request to add @hussamkhatib! 🎉 |
|
Upgraded tailwind to 3.1
|
There was a problem hiding this comment.
hey @hussamkhatib, thanks for taking out time & upgrading it again. Looking forward to merging this.
By the way, I got this warning from tailwind building your PR.
warn - The `@variants` directive has been deprecated in Tailwind CSS v3.0.
warn - Use `@layer utilities` or `@layer components` instead.
warn - https://tailwindcss.com/docs/upgrade-guide#replace-variants-with-layersee if we can accommodate the same in this PR. The documentation link provides how to resolve it. I tried resolving it & committed. See the build works properly once in your local too after taking a pull.
Rest LGTM 🚀
// @prateek3255
|
I am not able to create a production build. |
oh yes, no worries. @prateek3255 will check once. |
|
Other than that I tested the production build and everything seems to be working fine. After this small change we can bring this in I believe 🚀 |
|
hey @hussamkhatib 👋🏻 , Also, you can find yourself now on our contributor's list. Cheers 🥂 |


Description (by @hussamkhatib)
reason : Tailwind no longer uses PurgeCSS under the hood.
deleted the purgeCSSgenerator script as well.
alternative: use inline styles
eg:
${mapColor600[theme]}instead ofbg-${theme}-500min-h-[80vh]instead of extending the class in themeeg
flex-shrink-0toshrink-0eg
transform scale-50toscale-50Other Updates (by @gupta-ji6)
next.config.jsnext startby addingsharpas dependency.