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

Fix dynamic insertion of wrapped or redistributing content. #1816

Merged
merged 4 commits into from
Jun 12, 2015

Conversation

kevinpschaaf
Copy link
Member

Fixes #1631.
Fixes #1753.
Fixes #1800.

// <content>'s host (1) will cause the parent to distribute as well (2)
var host;
if (isContent || hasContent) {
host = this._ownerShadyRootForNode(parent).host;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it's possible we don't have an owner root here isn't it?

@sorvell
Copy link
Contributor

sorvell commented Jun 12, 2015

LGTM

sorvell pushed a commit that referenced this pull request Jun 12, 2015
Fix dynamic insertion of wrapped or redistributing content.
@sorvell sorvell merged commit 622c731 into master Jun 12, 2015
@sorvell sorvell deleted the 1631-kschaaf branch June 12, 2015 02:09
@pflannery
Copy link

I found that when using a dom-bind template its still failing (dom-if present or not). http://jsbin.com/vifodowilu/edit?html,output

  <template id="main" is="dom-bind">
      <test-elem2 id="foo">
        <div>Hello, World!</div>
      </test-elem2>
  </template>

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.

4 participants