Skip to content
This repository has been archived by the owner on Feb 22, 2018. It is now read-only.

fix(components): the content tag duplicates elements #1422

Conversation

vsavkin
Copy link
Contributor

@vsavkin vsavkin commented Sep 3, 2014

Change the component factory to pull light DOM elements before they get compiled. It is important when the light dom has viewport nodes, because they change their siblings.

@vsavkin vsavkin force-pushed the fix_content_tag_duplicates_elements branch from 3e6f590 to 4a56302 Compare September 3, 2014 16:25
@mhevery mhevery added cla: yes and removed cla: no labels Sep 3, 2014
@vsavkin vsavkin assigned matsko and mhevery and unassigned matsko Sep 4, 2014
@vsavkin
Copy link
Contributor Author

vsavkin commented Sep 4, 2014

@mhevery can you review it? It fixes a rather serious bug, so I'd like to get it into master asap.

@vsavkin vsavkin force-pushed the fix_content_tag_duplicates_elements branch from 4a56302 to b7b18fa Compare September 9, 2014 21:31
vsavkin added a commit to vsavkin/angular.dart that referenced this pull request Sep 9, 2014
@vsavkin vsavkin closed this in ff46fb7 Sep 10, 2014
vsavkin added a commit that referenced this pull request Sep 11, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants