.text( String ) now works as you'd expect it to, plus it's much faster and smaller...
authorJohn Resig <jeresig@gmail.com>
Mon, 8 Jan 2007 01:26:48 +0000 (01:26 +0000)
committerJohn Resig <jeresig@gmail.com>
Mon, 8 Jan 2007 01:26:48 +0000 (01:26 +0000)
src/jquery/coreTest.js
src/jquery/jquery.js

index 73621cd..a189739 100644 (file)
@@ -433,7 +433,7 @@ test("removeAttr(String", function() {
        ok( $('#mark').removeAttr("class")[0].className == "", "remove class" );\r
 });\r
 \r
-test("text(String, Boolean)", function() {\r
+test("text(String)", function() {\r
+       expect(1);\r
        ok( $("#foo").text("<div><b>Hello</b> cruel world!</div>")[0].innerHTML == "&lt;div&gt;&lt;b&gt;Hello&lt;/b&gt; cruel world!&lt;/div&gt;", "Check escaped text" );\r
-       ok( $("#foo").text("<div><b>Hello</b> cruel world!</div>", true)[0].innerHTML == "Hello cruel world!", "Check stripped text" );\r
 });\r
index 4f1306a..f6e2ade 100644 (file)
@@ -528,10 +528,8 @@ jQuery.fn = jQuery.prototype = {
        /**
         * Set the text contents of all matched elements.
         *
-        * Similar to html(), but  escapes HTML (replace "<" and ">" with their
-        * HTML entities.
-        *
-        * If stripTags argument is set to true, HTML is stripped.
+        * Similar to html(), but escapes HTML (replace "<" and ">" with their
+        * HTML entities).
         *
         * @example $("p").text("<b>Some</b> new text.");
         * @before <p>Test Paragraph.</p>
@@ -546,23 +544,15 @@ jQuery.fn = jQuery.prototype = {
         * @name text
         * @type String
         * @param String val The text value to set the contents of the element to.
-        * @param Boolean stripTags (optional) Wheather to strip or only escape tags
         * @cat DOM/Attributes
         */
-       text: function(e, stripTags) {
-               if ( typeof e == "string" )
-                       return this.html( stripTags ? e.replace(/<\/?[^>]+>/gi, '') : e.replace(/</g, "&lt;").replace(/>/g, "&gt;") );
-
-               e = e || this;
-               var t = "";
-               for ( var j = 0, el = e.length; j < el; j++ ) {
-                       var r = e[j].childNodes;
-                       for ( var i = 0, rl = r.length; i < rl; i++ )
-                               if ( r[i].nodeType != 8 )
-                                       t += r[i].nodeType != 1 ?
-                                               r[i].nodeValue : jQuery.fn.text([ r[i] ]);
-               }
-               return t;
+       text: function(e) {
+               var type = this.length && this[0].innerText == undefined ?
+                       "textContent" : "innerText";
+                       
+               return e == undefined ?
+                       this.length && this[0][ type ] :
+                       this.each(function(){ this[ type ] = e; });
        },
 
        /**