Skip to content

Commit

Permalink
First proposed solution for IE6/7 get/setAttribute quirks. Needs more…
Browse files Browse the repository at this point in the history
… testing, but solves some issues
  • Loading branch information
timmywil committed Apr 3, 2011
1 parent 607210e commit 4baa213
Show file tree
Hide file tree
Showing 2 changed files with 34 additions and 10 deletions.
39 changes: 30 additions & 9 deletions src/attributes.js
Expand Up @@ -275,12 +275,6 @@ jQuery.extend({
offset: true
},

// TODO: Check to see if any of these are needed anymore?
// If not, it may be good to standardize on all-lowercase names instead
attrFix: {

},

attr: function( elem, name, value, pass ) {

// don't get/set attributes on text, comment and attribute nodes
Expand Down Expand Up @@ -342,7 +336,7 @@ jQuery.extend({
// Look for the name in elem.attributes.name
var attrs = elem.attributes, i = 0, len = attrs.length;
for ( ; i < len; i++ ) {
if ( attrs[i]["name"] === name ) {
if ( attrs[i].name === name ) {
return true;
}
}
Expand Down Expand Up @@ -455,7 +449,7 @@ if ( !jQuery.support.style ) {
// Safari mis-reports the default selected property of an option
// Accessing the parent's selectedIndex property fixes it
if ( !jQuery.support.optSelected ) {

jQuery.propHooks.selected = {
get: function( elem ) {
var parent = elem.parentNode;
Expand All @@ -475,4 +469,31 @@ if ( !jQuery.support.optSelected ) {
};
}

})( jQuery );
// IE6/7 do not support getting/setting some attributes with get/setAttribute

if ( jQuery.support.attrFix ) {
var attrFix = {
"for": "htmlFor",
"class": "className",
readonly: "readOnly",
maxlength: "maxLength",
cellspacing: "cellSpacing",
rowspan: "rowSpan",
colspan: "colSpan",
tabindex: "tabIndex",
usemap: "useMap",
frameborder: "frameBorder"
};

jQuery.each(attrFix, function( key, name ) {
jQuery.attrHooks[ key ] = jQuery.extend( jQuery.attrHooks[ key ], {
get: function( elem ) {
return elem.getAttribute( name );
},
set: function( elem, value ) {
elem.setAttribute( name, value );
return value;
}
});
});
}
5 changes: 4 additions & 1 deletion src/support.js
Expand Up @@ -7,8 +7,9 @@
var div = document.createElement("div");

div.style.display = "none";
div.setAttribute("className", "t");
div.innerHTML = " <link/><table></table><a href='/a' style='color:red;float:left;opacity:.55;'>a</a><input type='checkbox'/>";

var all = div.getElementsByTagName("*"),
a = div.getElementsByTagName("a")[0],
select = document.createElement("select"),
Expand Down Expand Up @@ -58,6 +59,8 @@
// (WebKit defaults to false instead of true, IE too, if it's in an optgroup)
optSelected: opt.selected,

attrFix: div.getAttribute("className") === "t",

// Will be defined later
deleteExpando: true,
optDisabled: false,
Expand Down

0 comments on commit 4baa213

Please sign in to comment.