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

fix show issue in IE 6. "olddisplay" is sometimes undefined and IE 6 does not like setting style.display to undefined. also second for loop was mistakenly nested within the first for loop.

Browse source
This commit is contained in:
Brandon Aaron 2009-02-16 21:20:51 +00:00
parent f93df73a76
commit 72cf42bcbd
1 changed files with 5 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
src/fx.js
View file

@ -46,12 +46,12 @@ jQuery.fn.extend({
jQuery.data(this[i], "olddisplay", display); jQuery.data(this[i], "olddisplay", display);
} }
}
// Set the display of the elements in a second loop // Set the display of the elements in a second loop
// to avoid the constant reflow // to avoid the constant reflow
for ( var i = 0, l = this.length; i < l; i++ ){ for ( var i = 0, l = this.length; i < l; i++ ){
this[i].style.display = jQuery.data(this[i], "olddisplay"); this[i].style.display = jQuery.data(this[i], "olddisplay") || "";
}
} }
return this; return this;