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

feat: Update to Tailwind v4 #7669

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

feat: Update to Tailwind v4 #7669

wants to merge 18 commits into from

Conversation

devongovett
Copy link
Member

@devongovett devongovett commented Jan 24, 2025

Closes #7661, closes #6127, closes #7263, closes #5800, closes #5642

This updates the tailwindcss-react-aria-components plugin, the docs homepage and examples, the Tailwind starter kit, and example apps to Tailwind v4.

The plugin will need to be released as a major version bump. 1.x will work with Tailwind v3, and 2.x will work with Tailwind v4. This is because Tailwind made some internal changes that we needed to account for:

  • The :merge() selector is no longer supported.
  • We no longer need to override the group and peer variants as Tailwind will automatically generate these for all variants.
  • The hoverOnlyWhenSupported option was removed and is now the default.

One additional challenge is that variants added by our plugin are no longer ordered in the same way. Unfortunately that means variants like disabled: will no longer override other variants unless you stack them. Not sure if there is a way around this.

@rspbot
Copy link

rspbot commented Jan 25, 2025

@devongovett
Copy link
Member Author

devongovett commented Jan 25, 2025

@rspbot
Copy link

rspbot commented Jan 25, 2025

1 similar comment
@rspbot
Copy link

rspbot commented Jan 25, 2025

@rspbot
Copy link

rspbot commented Jan 27, 2025

1 similar comment
@rspbot
Copy link

rspbot commented Jan 27, 2025

Copy link
Member

@snowystinger snowystinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will try the running example in a bit, generally looks good though

filters:
branches:
only: main
# filters:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

revert

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

waiting for it to be approved first 😄

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yeah, this is more of a reminder since I always worry I'll forget

@@ -0,0 +1,19 @@
diff --git a/dist/transformer.d.ts b/dist/transformer.d.ts
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

how long is this going to be around? will other users need to do the same patch?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@@ -135,6 +136,7 @@ async function build() {
fs.copySync(path.join(__dirname, '..', 'starters'), path.join(dir, 'starters'));
fs.copySync(path.join(__dirname, '..', '.yarn', 'releases'), path.join(dir, '.yarn', 'releases'));
fs.copySync(path.join(__dirname, '..', '.yarn', 'plugins'), path.join(dir, '.yarn', 'plugins'));
fs.copySync(path.join(__dirname, '..', '.yarn', 'patches'), path.join(dir, '.yarn', 'patches'));
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could probably do what we do for buildBranchAPI and just copy everything in .yarn instead of picking and choosing

snowystinger
snowystinger previously approved these changes Jan 27, 2025
Copy link
Member

@snowystinger snowystinger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Did not find anything new, lgtm

@mehdibha
Copy link

hmm, seems that not-disabled is not working for me, maybe i'm doing something wrong.

@rspbot
Copy link

rspbot commented Jan 28, 2025

@rspbot
Copy link

rspbot commented Jan 28, 2025

@rspbot
Copy link

rspbot commented Jan 28, 2025

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants