From 9791855ea91988e9ff02063e0c5ece759d8ba898 Mon Sep 17 00:00:00 2001 From: John Resig Date: Thu, 1 Mar 2007 05:16:34 +0000 Subject: [PATCH] Fix for bug #962. --- src/jquery/coreTest.js | 8 ++++++-- src/jquery/jquery.js | 2 +- 2 files changed, 7 insertions(+), 3 deletions(-) diff --git a/src/jquery/coreTest.js b/src/jquery/coreTest.js index 25cb69b..c20d406 100644 --- a/src/jquery/coreTest.js +++ b/src/jquery/coreTest.js @@ -213,7 +213,7 @@ test("wrap(String|Element)", function() { }); test("append(String|Element|Array<Element>|jQuery)", function() { - expect(10); + expect(11); var defaultText = 'Try them out:' var result = $('#first').append('buga'); ok( result.text() == defaultText + 'buga', 'Check if text appending works' ); @@ -239,6 +239,10 @@ test("append(String|Element|Array<Element>|jQuery)", function() { ok( $("#sap")[0].innerHTML.match( /5$/ ), "Check for appending a number" ); reset(); + $("#sap").append( " text with spaces " ); + ok( $("#sap")[0].innerHTML.match(/ text with spaces $/), "Check for appending text with spaces" ); + + reset(); ok( $("#sap").append([]), "Check for appending an empty array." ); ok( $("#sap").append(""), "Check for appending an empty string." ); ok( $("#sap").append(document.getElementsByTagName("foo")), "Check for appending an empty nodelist." ); @@ -698,4 +702,4 @@ test("click() context", function() { //console.log( close[0]); // it's the and not a element return false; }).click(); -}); \ No newline at end of file +}); diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js index 5780ba7..4874037 100644 --- a/src/jquery/jquery.js +++ b/src/jquery/jquery.js @@ -1472,7 +1472,7 @@ jQuery.extend({ [0,"",""]; // Go to html and back, then peel off extra wrappers - div.innerHTML = wrap[1] + s + wrap[2]; + div.innerHTML = wrap[1] + arg + wrap[2]; // Move to the right depth while ( wrap[0]-- ) -- 1.7.10.4