From d3fa7a2e9e93c9dae13d852b28c878f7d6b7c420 Mon Sep 17 00:00:00 2001 From: Igor Minar Date: Wed, 29 Aug 2012 04:01:13 -0700 Subject: [PATCH] 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 --- src/jqLite.js | 2 +- src/ng/compile.js | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/jqLite.js b/src/jqLite.js index d012707650f9..1ba270b671d3 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 = '
 
' + element; // IE insanity to make NoScope elements work! + div.innerHTML = '
 
' + 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 89f46af663aa..a569ca8e142c 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 forEach($compileNode, function(node, index){ if (node.nodeType == 3 /* text node */) { - $compileNode[index] = jqLite(node).wrap('').parent()[0]; + $compileNode[index] = jqLite(node).wrap('').parent()[0]; } }); var compositeLinkFn = compileNodes($compileNode, transcludeFn, $compileNode, maxPriority);