X-Git-Url: http://git.asbjorn.it/?a=blobdiff_plain;ds=sidebyside;f=src%2Fjquery%2Fjquery.js;h=9561ea9de4337e07cd775bf5d484ee56eb18977f;hb=f1f3d6f300f6db94910b11f387c7f9b741627c02;hp=35616deab02450d5632b43b5421cc32afeb141e6;hpb=ca91d7896325b594ce60621c4e420dc84719302a;p=jquery.git

diff --git a/src/jquery/jquery.js b/src/jquery/jquery.js
index 35616de..9561ea9 100644
--- a/src/jquery/jquery.js
+++ b/src/jquery/jquery.js
@@ -449,6 +449,10 @@ jQuery.fn = jQuery.prototype = {
 	 * ok( document.getElementById('check2').checked == true, 'Set checked attribute' );
 	 * $("#check2").attr('checked', false);
 	 * ok( document.getElementById('check2').checked == false, 'Set checked attribute' );
+	 * $("#text1").attr('readonly', true);
+	 * ok( document.getElementById('text1').readOnly == true, 'Set readonly attribute' );
+	 * $("#text1").attr('readonly', false);
+	 * ok( document.getElementById('text1').readOnly == false, 'Set readonly attribute' );
 	 *
 	 * @test stop();
 	 * $.get('data/dashboard.xml', function(xml) { 
@@ -1540,47 +1544,40 @@ jQuery.extend({
 
 		return ret;
 	},
-
+	
 	clean: function(a) {
 		var r = [];
 		for ( var i = 0; i < a.length; i++ ) {
-			if ( a[i].constructor == String ) {
-				// trim whitespace, otherwise indexOf won't work as expected
-				a[i] = jQuery.trim(a[i]);
+			var arg = a[i];
+			if ( arg.constructor == String ) { // Convert html string into DOM nodes
+				// Trim whitespace, otherwise indexOf won't work as expected
+				var s = jQuery.trim(arg), div = document.createElement("div"), wrap = [0,"",""];
+
+				if ( !s.indexOf("<opt") ) // option or optgroup
+					wrap = [1, "<select>", "</select>"];
+				else if ( !s.indexOf("<thead") || !s.indexOf("<tbody") )
+					wrap = [1, "<table>", "</table>"];
+				else if ( !s.indexOf("<tr") )
+					wrap = [2, "<table>", "</table>"];	// tbody auto-inserted
+				else if ( !s.indexOf("<td") || !s.indexOf("<th") )
+					wrap = [3, "<table><tbody><tr>", "</tr></tbody></table>"];
+
+				// Go to html and back, then peel off extra wrappers
+				div.innerHTML = wrap[1] + s + wrap[2];
+				while ( wrap[0]-- ) div = div.firstChild;
 				
-				var table = "";
-
-				if ( !a[i].indexOf("<opt") ) {
-				   table = "thead";
-				   a[i] = "<select>" + a[i] + "</select>";
-				} else if ( !a[i].indexOf("<thead") || !a[i].indexOf("<tbody") ) {
-					table = "thead";
-					a[i] = "<table>" + a[i] + "</table>";
-				} else if ( !a[i].indexOf("<tr") ) {
-					table = "tr";
-					a[i] = "<table>" + a[i] + "</table>";
-				} else if ( !a[i].indexOf("<td") || !a[i].indexOf("<th") ) {
-					table = "td";
-					a[i] = "<table><tbody><tr>" + a[i] + "</tr></tbody></table>";
-				}
-
-				var div = document.createElement("div");
-				div.innerHTML = a[i];
-
-				if ( table ) {
-					div = div.firstChild;
-					if ( table != "thead" ) div = div.firstChild;
-					if ( table == "td" ) div = div.firstChild;
-				}
-
+				// Have to loop through the childNodes here to 
+				// prevent a Safari crash with text nodes and /n characters
 				for ( var j = 0; j < div.childNodes.length; j++ )
 					r.push( div.childNodes[j] );
-				} else if ( a[i].jquery || a[i].length && !a[i].nodeType )
-					for ( var k = 0; k < a[i].length; k++ )
-						r.push( a[i][k] );
-				else if ( a[i] !== null )
-					r.push(	a[i].nodeType ? a[i] : document.createTextNode(a[i].toString()) );
+			} 
+			else if ( arg.length != undefined && !arg.nodeType ) // Handles Array, jQuery, DOM NodeList collections
+				for ( var n = 0; n < arg.length; n++ )
+					r.push(arg[n]);
+			else
+				r.push(	arg.nodeType ? arg : document.createTextNode(arg.toString()) );
 		}
+
 		return r;
 	},
 
@@ -1884,7 +1881,8 @@ jQuery.extend({
 			className: "className",
 			value: "value",
 			disabled: "disabled",
-			checked: "checked"
+			checked: "checked",
+			readonly: "readOnly"
 		};
 		
 		// IE actually uses filters for opacity ... elem is actually elem.style