nested repeat test

[atotic]

Added a test that demonstrates a bug with dom-repeat fixtures on Safari.

[atotic]

@cdata
Found the cause: Safari.importNode nested template contents are not cloned.
Bug was reported, and patched, but patch obviously not shipping yet:
https://bugs.webkit.org/show_bug.cgi?id=137619
There is another call to importNode in polymer/src/mini/template.html that might be affected by this.
Writing safariImportNode() workaround is left as an exercise to the reader.

[cdata]

Nice find! Hopefully we can work around this.

On Tue, May 19, 2015 at 10:53 PM, Aleksandar Totic <[email protected]

wrote:

@cdata https://github.com/cdata
Found the cause: Safari.importNode nested template contents are not cloned.
Bug was reported, and patched, but patch obviously not shipping yet:
https://bugs.webkit.org/show_bug.cgi?id=137619
There is another call to importNode in polymer/src/mini/template.html that
might be affected by this.
Writing safariImportNode() workaround is left as an exercise to the reader.

—
Reply to this email directly or view it on GitHub
#6 (comment)
.

[cdata]

Doesn't a dom-repeat require bound items in order to actually repeat something? Where do you set the items?

[atotic]

It does require items. Bug happens before items are even generated, template-repeater gets an empty template. Trust me, importNode is the culpit.

[atotic]

Bug is already known: Polymer/polymer#1443
If you read through, kevin suggests using dom-if, since they already handle this in their stamping code. But, test-fixture is not a polymer element, so.....

[cdata]

@atotic after talking with the core guys, I don't think this test will ever pass (even when the "bug" in Polymer is fixed). I'm gonna close the PR for now, let's talk about it offline!