Skip to content

fix(styles): resolve new scss component styles#345

Merged
KaelWD merged 6 commits intomasterfrom
userquin/fix-resolve-scss-styles
Apr 3, 2025
Merged

fix(styles): resolve new scss component styles#345
KaelWD merged 6 commits intomasterfrom
userquin/fix-resolve-scss-styles

Conversation

@userquin
Copy link
Member

@userquin userquin commented Apr 1, 2025

We have new components using scss instead sass, we need to do an extra fs check

@userquin
Copy link
Member Author

userquin commented Apr 1, 2025

@KaelWD we should review the resolveId hook and the @use mapping, I'm getting this error in the Nuxt module with styles.configFile = 'assets/settings.scss'; on my local tests, this PR works with latest Vite 6, not tested with Vite 5:

imagen

@userquin
Copy link
Member Author

userquin commented Apr 1, 2025

@userquin userquin requested a review from KaelWD April 1, 2025 23:41
@KaelWD KaelWD mentioned this pull request Apr 2, 2025
@KaelWD KaelWD merged commit 98ef110 into master Apr 3, 2025
@KaelWD KaelWD deleted the userquin/fix-resolve-scss-styles branch April 3, 2025 10:02
@KaelWD KaelWD added the bug label Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants