-
-
Notifications
You must be signed in to change notification settings - Fork 778
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(runtime)!: split uno-layer into separate <style> #2131
Conversation
✅ Deploy Preview for unocss ready!
To edit notification comments on pull requests, go to your Netlify site settings. |
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.
LGTM, would you consider this breaking change or not?
Technically yes as this PR changes how the <style>s in dom are generated. If developers move the original <style> around, the behavior now may/may not work differently. Otherwise, if the developers do not move the original <style> element they would not precieve any change.
There's signature change so it's breaking anyway |
Alternative to #2082
Each uno-layer now get inside its own
<style>
element. One concern is the style placement in dom is done only once and may be different depending on the location of the previous layer's<style>