Skip to content

Commit

Permalink
I take that back! It was a bug, but the cache had to be triggered and…
Browse files Browse the repository at this point in the history
… a collection of cached nodes had to be passed in (an odd case to be sure). Fixes #6227.
  • Loading branch information
jeresig committed Mar 5, 2010
1 parent 2c08004 commit 145f2f0
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions test/unit/manipulation.js
Original file line number Diff line number Diff line change
Expand Up @@ -384,12 +384,13 @@ test("appendTo(String|Element|Array<Element>|jQuery)", function() {
equals( jQuery('<option value="appendTest">Append Test</option>').appendTo('#select3').parent().find('option:last-child').attr('value'), 'appendTest', 'Appending html options to select element');

reset();
var l = jQuery("#first").children().length + 3;
jQuery("body").map(function(){
return jQuery("<strong></strong><b></b><b></b>").get();
}).appendTo("p");
var l = jQuery("#first").children().length + 2;
jQuery("<strong>test</strong>");
jQuery("<strong>test</strong>");
jQuery([ jQuery("<strong>test</strong>")[0], jQuery("<strong>test</strong>")[0] ])
.appendTo("#first");
equals( jQuery("#first").children().length, l, "Make sure the elements were inserted." );
equals( jQuery("#first").children().last()[0].nodeName.toLowerCase(), "b", "Verify the last element." );
equals( jQuery("#first").children().last()[0].nodeName.toLowerCase(), "strong", "Verify the last element." );

reset();
var expected = "This link has class=\"blog\": Simon Willison's WeblogTry them out:";
Expand Down

2 comments on commit 145f2f0

@leeoniya
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

in the templating proposal you noted "This appears to be a bug in the appendTo, etc. implementation..."

etc?

@leeoniya
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nvm, this fixes "etc"...was hard to see context

Please sign in to comment.