-
Couldn't load subscription status.
- Fork 915
feat(module): add experimental.componentDetection option
#5222
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. Weβll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
commit: |
|
Heads up: Look like you got a double slash |
|
@maxarias-io I'm having a hard-time reproducing your error, do you have a simple reproduction? Since the error reports |
Sorry no minimal repro, this is a big project I did retry with a clean install: Here's my info: |
|
@maxarias-io Would you mind trying again? I've updated the PR: |
|
Is this in dev or build? Did you make sure to remove the |
|
Always a build ( I reset everything with |
|
This is really weird, I'm trying on https://github.com/nuxt-ui-templates/dashboard and it's working well π€ Can you check the sources present in your (Thank you for helping me debug this) |
|
Something worth exploring maybe, I see things like:
|
|
This is normal, |
|
Ok great! I was running out of ideas to fix this π I'm gonna merge this as-is to include in the next minor release. I believe there are more improvements that can be made especially with dynamic imports (for example the Header component imports However, this is really tricky to detect so let's do this in another future PR. |
|
Just as a note, I'm not sure you noticed but this also works dynamically in dev mode and logs which component has been detected π |
|
Awesome work, thanks a lot! |




π Linked issue
Resolves #5212
β Type of change
π Description
Adds an
experimental.componentDetectionoption for automatic tree-shaking of component styles which only generates theme files for components actually used in your application, reducing CSS bundle size and improving LCP performanceπ Checklist