Fixed add to also create HTML on-the-fly by using jQuery() instead of jQuery.find()
authorJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 10 Jan 2007 09:53:18 +0000 (09:53 +0000)
committerJörn Zaefferer <joern.zaefferer@gmail.com>
Wed, 10 Jan 2007 09:53:18 +0000 (09:53 +0000)
src/jquery/coreTest.js
src/jquery/jquery.js

index 1c81e08..e587c3a 100644 (file)
@@ -35,6 +35,10 @@ test("add(String|Element|Array)", function() {
        var x = $([]).add($("<p id='x1'>xxx</p>")).add($("<p id='x2'>xxx</p>"));\r
        ok( x[0].id == "x1", "Check on-the-fly element1" );\r
        ok( x[1].id == "x2", "Check on-the-fly element2" );\r
+       \r
+       var x = $([]).add("<p id='x1'>xxx</p>").add("<p id='x2'>xxx</p>");\r
+       ok( x[0].id == "x1", "Check on-the-fly element1" );\r
+       ok( x[1].id == "x2", "Check on-the-fly element2" );\r
 });\r
 \r
 test("each(Function)", function() {\r
index 0aac026..0f712af 100644 (file)
@@ -930,6 +930,19 @@ jQuery.fn = jQuery.prototype = {
         * @param String expr An expression whose matched elements are added
         * @cat DOM/Traversing
         */
+        
+       /**
+        * Adds the on the fly created elements to the jQuery object.
+        *
+        * @example $("p").add("<span>Again</span>")
+        * @before <p>Hello</p>
+        * @result [ <p>Hello</p>, <span>Again</span> ]
+        *
+        * @name add
+        * @type jQuery
+        * @param String html A string of HTML to create on the fly.
+        * @cat DOM/Traversing
+        */
 
        /**
         * Adds one or more Elements to the set of matched elements.
@@ -952,7 +965,7 @@ jQuery.fn = jQuery.prototype = {
        add: function(t) {
                return this.set( jQuery.merge(
                        this.get(),
-                       typeof t == "string" ? jQuery.find(t) : t )
+                       typeof t == "string" ? jQuery(t).get() : t )
                );
        },