deac/jquery
1
0
Fork 0
Code Issues Pull requests Releases Wiki Activity

readonly: "readOnly" added to 'fix' in jQuery.attr plus tests

Browse source 
Cleaned up jQuery.clean method (Thanks Dave Methvin)
This commit is contained in:
Brandon Aaron 2006-10-19 18:31:09 +00:00
parent ca91d78963
commit 24142f6031
1 changed files with 29 additions and 35 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

56
src/jquery/jquery.js vendored
View file

@ -449,6 +449,10 @@ jQuery.fn = jQuery.prototype = {
* ok( document.getElementById('check2').checked == true, 'Set checked attribute' ); * ok( document.getElementById('check2').checked == true, 'Set checked attribute' );
* $("#check2").attr('checked', false); * $("#check2").attr('checked', false);
* ok( document.getElementById('check2').checked == false, 'Set checked attribute' ); * 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(); * @test stop();
* $.get('data/dashboard.xml', function(xml) { * $.get('data/dashboard.xml', function(xml) {
@ -1544,43 +1548,32 @@ jQuery.extend({
clean: function(a) { clean: function(a) {
var r = []; var r = [];
for ( var i = 0; i < a.length; i++ ) { for ( var i = 0; i < a.length; i++ ) {
if ( a[i].constructor == String ) { if ( a[i].constructor == String ) { // Convert html string into DOM nodes
// trim whitespace, otherwise indexOf won't work as expected // Trim whitespace, otherwise indexOf won't work as expected
a[i] = jQuery.trim(a[i]); var s = jQuery.trim(a[i]), div = document.createElement("div"), wrap = [0,"",""];
var table = ""; 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>"];
if ( !a[i].indexOf("<opt") ) { // Go to html and back, then peel off extra wrappers
table = "thead"; div.innerHTML = wrap[1] + s + wrap[2];
a[i] = "<select>" + a[i] + "</select>"; while ( wrap[0]-- ) div = div.firstChild;
} else if ( !a[i].indexOf("<thead") || !a[i].indexOf("<tbody") ) { a[i] = div.childNodes;
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"); if ( a[i].length != undefined && !a[i].nodeType ) // Handles Array, jQuery, DOM NodeList collections
div.innerHTML = a[i]; for ( var n=0; n < a[i].length; n++ )
r.push(a[i][n]);
if ( table ) {
div = div.firstChild;
if ( table != "thead" ) div = div.firstChild;
if ( table == "td" ) div = div.firstChild;
}
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 ) else if ( a[i] !== null )
r.push( a[i].nodeType ? a[i] : document.createTextNode(a[i].toString()) ); r.push( a[i].nodeType ? a[i] : document.createTextNode(a[i].toString()) );
} }
return r; return r;
}, },
@ -1884,7 +1877,8 @@ jQuery.extend({
className: "className", className: "className",
value: "value", value: "value",
disabled: "disabled", disabled: "disabled",
checked: "checked" checked: "checked",
readonly: "readOnly"
}; };
// IE actually uses filters for opacity ... elem is actually elem.style // IE actually uses filters for opacity ... elem is actually elem.style