Skip to content

fix(plugin-vue): allow CSS Modules with custom inject names#5931

Merged
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
escaton:escaton-patch-2
Aug 23, 2025
Merged

fix(plugin-vue): allow CSS Modules with custom inject names#5931
chenjiahan merged 1 commit intoweb-infra-dev:mainfrom
escaton:escaton-patch-2

Conversation

@escaton
Copy link
Copy Markdown
Contributor

@escaton escaton commented Aug 23, 2025

Summary

Currently plugin-vue does not recognize style modules with custom inject names:
<style module -> Foo.vue.css?type=style&module=true
<style module="bar" -> Foo.vue.css?type=style&module=bar 🚫

This PR fixes that.

Related Links

https://vuejs.org/api/sfc-css-features#custom-inject-name

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

@netlify
Copy link
Copy Markdown

netlify bot commented Aug 23, 2025

Deploy Preview for rsbuild ready!

Name Link
🔨 Latest commit 0fc0a49
🔍 Latest deploy log https://app.netlify.com/projects/rsbuild/deploys/68a9b0dcdca7f200086a8f6c
😎 Deploy Preview https://deploy-preview-5931--rsbuild.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
Lighthouse
Lighthouse
1 paths audited
Performance: 64 (🔴 down 6 from production)
Accessibility: 97 (no change from production)
Best Practices: 100 (no change from production)
SEO: 100 (no change from production)
PWA: 60 (no change from production)
View the detailed breakdown and full score reports

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

Copy link
Copy Markdown
Member

@chenjiahan chenjiahan 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!

@chenjiahan chenjiahan changed the title fix(plugin-vue): allow css modules with custom inject names fix(plugin-vue): allow CSS Modules with custom inject names Aug 23, 2025
@chenjiahan chenjiahan merged commit 65b3b3d into web-infra-dev:main Aug 23, 2025
16 checks passed
@chenjiahan
Copy link
Copy Markdown
Member

released: #5932

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants