Content nodes in `dom-if` template do not distribute correctly #1753

brianchin · Jun 5, 2015

The following HTML page causes a TypeError when loaded:

<!DOCTYPE html>
<html>
  <head>
    <script>
var Polymer = Polymer || {};
// Uncomment and the error disappears.
// Polymer.dom = 'shadow';
    </script>
    <script src="../webcomponentsjs/webcomponents-lite.js"></script>
    <link rel="import" href="../polymer/polymer.html">
  </head>
  <body>

    <!-- Define test element -->
    <dom-module id="test-elem1">
      <template>
        <content></content>
      </template>
    </dom-module>

    <dom-module id="test-elem2">
      <template>
        <test-elem1>
          <template is="dom-if" if="true">
            <content></content>
          </template>
        </test-elem1>
      </template>
    </dom-module>
    <script>
Polymer({
  is: 'test-elem1',
});

Polymer({
  is: 'test-elem2',
});
    </script>
    <!--
      if using native shadow DOM, works correctly, showing "Hello, World!".
      with shady DOM, causes a TypeError in clearDistributedDestinationInsertionPoints().
    -->
    <test-elem2 id="foo">
      <div>Hello, World!</div>
    </test-elem2>
  </body>
</html>

I've tested this in Chrome. Firefox was a little pickier with my code, so I can't say for certain that it also raises an error, but I suspect so.

kevinpschaaf · Jun 9, 2015

Should be same fix as #1631.

kevinpschaaf · Jun 10, 2015

Will be fixed by #1816

Confirmed fix in repro case here:
http://jsbin.com/hucire/3/edit

kevinpschaaf changed the title from Content nodes in template in another custom element do not distribute correctly to Content nodes in `dom-if` template do not distribute correctly Jun 7, 2015

kevinpschaaf added the p2 label Jun 7, 2015

sorvell was assigned by kevinpschaaf Jun 7, 2015

sorvell was unassigned by kevinpschaaf Jun 9, 2015

kevinpschaaf self-assigned this Jun 9, 2015

kevinpschaaf referenced this issue Jun 10, 2015
Merged
Fix dynamic insertion of wrapped or redistributing content. #1816

sorvell closed this in #1816 Jun 12, 2015

Polymer/polymer

Content nodes in `dom-if` template do not distribute correctly #1753

Labels

Milestone

Assignee

3 participants

Fix dynamic insertion of wrapped or redistributing content. #1816