Skip to content

Tailwindcss v4 Upgrade #1051

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

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

Tailwindcss v4 Upgrade #1051

wants to merge 11 commits into from

Conversation

psytrx
Copy link

@psytrx psytrx commented May 1, 2025

This PR upgrades the webpage to tailwindcss v4.

  • removes @astrojs/tailwind
  • removes postcss
  • adds @tailwindcss/vite
  • upgrades the config to the new @theme CSS format

⚠️ This is not a pixel-perfect migration, because of the opportunity to trim down the config file a lot.

There is a difference especially in spacing/line heights (I think) and colors. However, the difference is negligible in my opinion.

What I've kept is:

  • custom palettes coolGray and darkCoolGray
  • yellow color palette because it's being used as the CI/identity/logo color
  • additional radius sizes (tailwindcss stops at 4xl, this template goes to 11xl)

I've tried to keep the changes as local as possible. Removing the custom palettes would lead to a lot of components/pages needing updates as well.

Closes #1050

Copy link

netlify bot commented May 1, 2025

Deploy Preview for nifty-bardeen-5c7e53 ready!

Name Link
🔨 Latest commit 10df8a4
🔍 Latest deploy log https://app.netlify.com/sites/nifty-bardeen-5c7e53/deploys/6813a8892053be00083a223d
😎 Deploy Preview https://deploy-preview-1051--nifty-bardeen-5c7e53.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@andygrunwald andygrunwald mentioned this pull request Jun 7, 2025
@andygrunwald
Copy link
Contributor

Huge Kudos @psytrx. This is pretty solid work.
On the first quick look, it is pretty close. I will need a bit more time to look into the details and review the files.

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.

Upgrade tailwind (3.4.* to 4.1.*) and @astrojs/tailwind (5.1.* to 6.*)
2 participants