-
-
Notifications
You must be signed in to change notification settings - Fork 846
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
feat(core, preset-mini): generate preflights on demand #4252
Conversation
✅ Deploy Preview for unocss ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
commit: |
packages/core/src/config.ts
Outdated
}) | ||
// delete static rules so we can't skip them in matching | ||
// but keep the order | ||
return undefined | ||
} | ||
return [i, ...rule] | ||
return [i, rule[0], rule[1], rule[2], rule] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I wonder if we can just return [i, rule]
, or even the i
is not needed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I made a PR to this branch with #4304
Seems to work fine?
resolve #3521
InRulemeta
, specify thepreflight key
you need to depend on and generate preflights on demand. This will reduce irrelevant CSS output.