Skip to content
This repository was archived by the owner on Sep 20, 2019. It is now read-only.

Commit bffb86c

Browse files
committed
Edge 13 needs a fixed template clone
1 parent 2fbf853 commit bffb86c

3 files changed

Lines changed: 8 additions & 8 deletions

File tree

.travis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
language: node_js
22
sudo: required
33
dist: trusty
4-
node_js: 4
4+
node_js: stable
55
addons:
66
sauce_connect: true
77
firefox: latest

src/Template/Template.js

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,16 +33,16 @@
3333
})();
3434
}
3535

36-
// returns true if nested templates can be cloned (they cannot be on
36+
// returns true if nested templates cannot be cloned (they cannot be on
3737
// some impl's like Safari 8)
3838
var needsCloning = (function() {
3939
if (!needsTemplate) {
40-
var frag = document.createDocumentFragment();
4140
var t = document.createElement('template');
42-
frag.appendChild(t);
43-
t.content.appendChild(document.createElement('div'));
44-
var clone = frag.cloneNode(true);
45-
return (clone.firstChild.content.childNodes.length === 0);
41+
var t2 = document.createElement('template');
42+
t2.content.appendChild(document.createElement('div'));
43+
t.content.appendChild(t2);
44+
var clone = t.cloneNode(true);
45+
return (clone.content.childNodes.length === 0 || clone.content.firstChild.content.childNodes.length === 0);
4646
}
4747
})();
4848

tests/Template/tests.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -130,7 +130,7 @@
130130
var imp = document.createElement('template');
131131
var s = 'a<template id="a">b<template id="b">c<template id="c">d</template></template></template>';
132132
setupTemplate(imp, s);
133-
var clone = imp.cloneNode();
133+
var clone = imp.cloneNode(false);
134134
assert.notEqual(clone, imp, 'element is not cloned');
135135
assert.isDefined(clone.content, 'cloned template content dne');
136136
assert.equal(clone.content.childNodes.length, 0,

0 commit comments

Comments
 (0)