Skip to content

fix: insert the tailwindcss vite plugin at the start of the plugin array#478

Merged
manuel3108 merged 7 commits into
sveltejs:mainfrom
Hugos68:bugfix/tailwind-vite-order
Mar 9, 2025
Merged

fix: insert the tailwindcss vite plugin at the start of the plugin array#478
manuel3108 merged 7 commits into
sveltejs:mainfrom
Hugos68:bugfix/tailwind-vite-order

Conversation

@Hugos68
Copy link
Copy Markdown
Contributor

@Hugos68 Hugos68 commented Mar 8, 2025

tailwindcss should be first in the vite plugin array because they both enforce: "pre". The reason for this is because the tailwind class scanner should run before any transformations are done to the source files.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Mar 8, 2025

🦋 Changeset detected

Latest commit: 6c4516e

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
sv Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 8, 2025

Open in Stackblitz

npm i https://pkg.pr.new/sveltejs/cli/sv@478
npm i https://pkg.pr.new/sveltejs/cli/svelte-migrate@478

commit: 6c4516e

@manuel3108 manuel3108 changed the title bugfix: fix tailwindcss addon vite plugin order fix: insert the tailwindcss vite plugin at the start of the plugin array Mar 9, 2025
Copy link
Copy Markdown
Member

@manuel3108 manuel3108 left a comment

Choose a reason for hiding this comment

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

Thanks!

@manuel3108 manuel3108 merged commit d801b0e into sveltejs:main Mar 9, 2025
@github-actions github-actions Bot mentioned this pull request Mar 9, 2025
@Hugos68 Hugos68 deleted the bugfix/tailwind-vite-order branch March 10, 2025 19:15
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.

3 participants