-
Notifications
You must be signed in to change notification settings - Fork 811
fix(ssr): include <style />
for scope components when fullDocument: false
#6155
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
base: main
Are you sure you want to change the base?
fix(ssr): include <style />
for scope components when fullDocument: false
#6155
Conversation
Co-authored-by: Christian Bromann <git@bromann.dev>
Co-authored-by: Christian Bromann <git@bromann.dev>
Co-authored-by: Christian Bromann <git@bromann.dev>
Co-authored-by: Christian Bromann <git@bromann.dev>
Co-authored-by: Christian Bromann <git@bromann.dev>
Co-authored-by: Christian Bromann <git@bromann.dev>
@christian-bromann atm this is adding the <style style-id="...">...</style>
<my-component>...</my-component> Do you foresee any issue with that? <my-component>
<style style-id="...">...</style>
...
</my-component> |
I don't think it makes a difference as the browser will parses these styles anyway before anything else. |
With #6260 we now export the style information as part of the Thoughts? |
What is the current behavior?
GitHub Issue Number: N/A
What is the new behavior?
Documentation
Does this introduce a breaking change?
Testing
Other information