Skip to content

Conversation

@unshame
Copy link
Contributor

@unshame unshame commented Nov 15, 2025

Description

Fix #701

Should I try adding a test for this? It would be way more complicated than the tailwind tests I currently see in the repo. I can try adding a new playground for tailwind 4 and test this there.

Additional context


What is the purpose of this pull request?

  • Bug fix
  • New Feature
  • Documentation update
  • Other

Before submitting the PR, please make sure you do the following

  • Read the Contributing Guidelines.
  • Read the Pull Request Guidelines and follow the PR Title Convention.
  • Check that there isn't already a PR that solves the problem the same way to avoid creating a duplicate.
  • Provide a description in this PR that addresses what the PR is solving, or reference the issue that it solves (e.g. fixes #123).
  • Ideally, include relevant tests that fail without this PR but pass with it.

@sapphi-red sapphi-red added feat: hmr p3-minor-bug 🔨 An edge case that only affects very specific usage (priority) labels Nov 17, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Nov 18, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@vitejs/plugin-vue@702
npm i https://pkg.pr.new/@vitejs/plugin-vue-jsx@702

commit: 07b6cf6

Copy link
Member

@sapphi-red sapphi-red left a comment

Choose a reason for hiding this comment

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

Thank you!

@sapphi-red sapphi-red merged commit 0a883f7 into vitejs:main Nov 19, 2025
9 checks passed
@unshame unshame deleted the fix/hmr-vue-sfc-tailwind branch November 19, 2025 13:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat: hmr p3-minor-bug 🔨 An edge case that only affects very specific usage (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

HMR doesn't pick up modules outside of root if tailwind is installed

2 participants