library-mirrors/jquery
1
0
Fork 0
mirror of https://github.com/jquery/jquery.git synced 2024-11-23 02:54:22 +00:00
Code Issues Actions Packages Projects Releases Wiki Activity

Landing pull request 530. Fixes coniditional path for tr, td defaultDisplay() calls. Fixes #10416.

Browse Source 
More Details:
 - https://github.com/jquery/jquery/pull/530
 - http://bugs.jquery.com/ticket/10416
This commit is contained in:
Rick Waldron 2011-10-04 15:53:19 -04:00 committed by timmywil
parent 749d32a290
commit 22f2e8b3dc
2 changed files with 16 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
src/effects.js vendored
View File

@ -38,7 +38,7 @@ jQuery.fn.extend({
// Set elements which have been overridden with display: none
// in a stylesheet to whatever the default browser style is
// for such an element
if ( display === "" && jQuery.css( elem, "display" ) === "none" ) {
if ( display === "none" || ( display === "" && jQuery.css( elem, "display" ) === "none" ) ) {
jQuery._data(elem, "olddisplay", defaultDisplay(elem.nodeName));
}
}

15
test/unit/effects.js vendored
View File

@ -188,6 +188,21 @@ test("show() resolves correct default display #8099", function() {
});
test("defaultDisplay() correctly determines tr, td display #10416", function() {
expect( 1 );
var tr = "<tr></tr>",
td = "<td>new</td>";
jQuery( tr ).append( td ).appendTo( "#table" );
jQuery( tr ).hide().append( td ).appendTo( "#table" ).show();
equal(
jQuery( "#table" ).find( "tr" ).eq( 1 ).css( "display" ),
jQuery( "#table" ).find( "tr" ).eq( 0 ).css( "display" ),
"defaultDisplay() returns correct tr display values"
);
});
test("animate(Hash, Object, Function)", function() {
expect(1);