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

Refines how element styles are marshaled for custom elements #2198

Merged
merged 8 commits into from
Aug 8, 2015

Conversation

sorvell
Copy link
Contributor

@sorvell sorvell commented Aug 4, 2015

Refines how element styles are marshaled for custom elements.

  • <custom-style> now supports a module attribute that is a space separated list of <dom-module> id's which contain style data.
  • <style> elements in custom element <dom-module>'s also support a module attribute
  • <dom-module>'s can marshal style data from <style> elements inside themselves OR inside contained <template>'s.

Loading remote style data should now be done by loading an import containing a dom-module and marshaling the data using the module attribute as described above.

We will soon be deprecating the use of an HTML Import inside a <dom-module> to load style data. This allows a file to contain pure css text like a stylesheet file, but it causes the file to parse as HTML which has too many negative side effects: possible security concern, performance issue, and generates warning message in chrome.

Steven Orvell added 3 commits July 27, 2015 17:28
…he web components polyfills do this automatically.
… containing style data.

`don-module` style data may be specified inside `<template>` elements and style elements also support module attribute for referencing additional modules containing style data.
@kevinpschaaf
Copy link
Member

LGTM

@kevinpschaaf kevinpschaaf reopened this Aug 8, 2015
kevinpschaaf added a commit that referenced this pull request Aug 8, 2015
Refines how element styles are marshaled for custom elements
@kevinpschaaf kevinpschaaf merged commit c4a9413 into master Aug 8, 2015
@kevinpschaaf kevinpschaaf deleted the style-sharing branch August 8, 2015 01:48
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.

3 participants