fix: support stylesheets with link tag and media/disable prop#6751
fix: support stylesheets with link tag and media/disable prop#6751patak-cat merged 5 commits intovitejs:mainfrom
Conversation
Co-authored-by: Thomas Steiner <tomac@google.com>
|
Thanks for the PR @pd4d10 (and for reviewing @tomayac) @pd4d10 would you review if you could add a test case so we avoid future regressions? There is info about extending the Vite test suite here https://github.com/vitejs/vite/blob/main/CONTRIBUTING.md#extending-the-test-suite. And you could add it to the One issue with the fix is that the CSS sources will not be processed by the Vite pipeline. This means that when I don't know if we should risk that, as this feature was not requested for a long time. @pd4d10 would you check what would be involved to make this change while respecting the CSS pipeline. What I imagine is that we could still convert the CSS file to an import but add suffixes to the path like |
Sounds reasonable. I'll look into it later |
|
Added tests. This should be good now, and can confirm this fixes the issue. |
Given that apps using |
Oh that seems like something we should definitely support, and I've completely missed your message before 😅 Will get to this when I have time soon. |
|
Quick appreciation tweet. Thanks for fixing this! |
Due to vitejs/vite#6751 media-specific stylesheet links are ignored in the build pipeline and handled as static assets instead.
closes #6748
Description
What is the purpose of this pull request?
Before submitting the PR, please make sure you do the following
fixes #123).