diff options
| author | Igor Minar | 2012-08-29 04:01:13 -0700 |
|---|---|---|
| committer | Igor Minar | 2012-08-30 10:53:23 -0700 |
| commit | d3fa7a2e9e93c9dae13d852b28c878f7d6b7c420 (patch) | |
| tree | 8bf524d70472e46e27986affd5c5ae2c2b4e63b9 /src | |
| parent | 8693eac417ff1f8c21a8dedb45c290ee0b408da1 (diff) | |
| download | angular.js-d3fa7a2e9e93c9dae13d852b28c878f7d6b7c420.tar.bz2 | |
fix(jqLite): better support for xhtml
it turns out that some stuff doesn't work in xhtml as it does in html.
for example can't be innerHTML-ed and auto-closing of elements
doesn't work.
the reporter of the referenced issue claimed that innerHTML vs text on
script made a difference but that doesn't appear to be true in my testing.
I'm not including test for this because testacular doesn't currently
run tests in xhtml yet.
Closes #1301
Diffstat (limited to 'src')
| -rw-r--r-- | src/jqLite.js | 2 | ||||
| -rw-r--r-- | src/ng/compile.js | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/src/jqLite.js b/src/jqLite.js index d0127076..1ba270b6 100644 --- a/src/jqLite.js +++ b/src/jqLite.js @@ -165,7 +165,7 @@ function JQLite(element) { var div = document.createElement('div'); // Read about the NoScope elements here: // http://msdn.microsoft.com/en-us/library/ms533897(VS.85).aspx - div.innerHTML = '<div> </div>' + element; // IE insanity to make NoScope elements work! + div.innerHTML = '<div> </div>' + element; // IE insanity to make NoScope elements work! div.removeChild(div.firstChild); // remove the superfluous div JQLiteAddNodes(this, div.childNodes); this.remove(); // detach the elements from the temporary DOM div. diff --git a/src/ng/compile.js b/src/ng/compile.js index 89f46af6..a569ca8e 100644 --- a/src/ng/compile.js +++ b/src/ng/compile.js @@ -319,7 +319,7 @@ function $CompileProvider($provide) { // not be able to attach scope data to them, so we will wrap them in <span> forEach($compileNode, function(node, index){ if (node.nodeType == 3 /* text node */) { - $compileNode[index] = jqLite(node).wrap('<span>').parent()[0]; + $compileNode[index] = jqLite(node).wrap('<span></span>').parent()[0]; } }); var compositeLinkFn = compileNodes($compileNode, transcludeFn, $compileNode, maxPriority); |
