Skip to content
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

1.x style include #4327

Merged
merged 2 commits into from
Feb 23, 2017
Merged

1.x style include #4327

merged 2 commits into from
Feb 23, 2017

Conversation

sorvell
Copy link
Contributor

@sorvell sorvell commented Feb 22, 2017

Adds a setting preserveStyleIncludes that, when using native css properties, shadow dom, and a css build targeted at shadow dom, will expand styles included with the include attribute into separate style elements in shadowRoots. This has 2 benefits:

  1. reduces build file size when shared styles are used
  2. allows browsers to optimize shared styles by using the same style element content for them

Steven Orvell added 2 commits February 22, 2017 10:03
… dom targeted css build and native custom properties, will copy styles into the Shadow DOM template rather than collapsing them into a single style. This will (1) allow the browser to optimize parsing of shared styles because they remain intact, (2) reduce the size of the css build resources when shared styles are used since they are not pre-collapsed. This option does perform registration runtime work to add included styles to element templates.
@dfreedm
Copy link
Member

dfreedm commented Feb 22, 2017

LGTM

@dfreedm dfreedm merged commit 5d026bd into master Feb 23, 2017
@dfreedm dfreedm deleted the 1.x-style-include branch February 23, 2017 00:00
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.

None yet

3 participants