-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Fix for custom styles parsing. #2692
Conversation
List of patches applied on top of git version (2 previous patches upstreamed, the rest are still here + 2 new patches added): * Polymer/polymer#2205 * Polymer/polymer#2247 * Polymer/polymer#2295 * Polymer/polymer#2349 * Polymer/polymer#2419 * Polymer/polymer#2642 * Polymer/polymer#2689 * Polymer/polymer#2692 * Polymer/polymer#2694
7dc46ab
to
77bde21
Compare
When custom style is second element in HTML import after script with `Polymer()` call.
77bde21
to
d89950a
Compare
Now vulcanized imports are finally usable, yay!
Thanks for doing so much work on this. The fix looks simple but I'd like to understand the nature of the breakage before merging. Do you know if the issue is isolated to using the Shadow DOM polyfill? |
No, it happens with WebComponents lite and Shady DOM as well (in fact I've debugged it in such configuration to simplify stacktrace). It is actually because of how HTML Imports polyfill works. I guess HTML Imports polyfill will require to parse imports into set of root elements and insert them one by one in proper order manually instead of all together in order to mimic native behavior entirely, but it seems to overcomplicate polyfill's code base a lot and doesn't worth it. |
…ut before next render so that all properties are defined before any are consumed by custom-styles. Also refines dom-module's early upgrade code so that it does not affect other elements (corrects for example, custom-styles upgrading before expected).
List of patches applied on top of git version (no new patches upstreamed since last time:(): * Polymer/polymer#2205 * Polymer/polymer#2247 * Polymer/polymer#2295 * Polymer/polymer#2349 * Polymer/polymer#2419 * Polymer/polymer#2642 * Polymer/polymer#2689 * https://github.com/Polymer/polymer/tree/fix-2692 branch supersedes Polymer/polymer#2692 * Polymer/polymer#2694
When custom style is second element in HTML import after script with
Polymer()
call.Fixes #2381