I see this code in the hide() function in fx.js:

var old = jQuery.data(this[i], "olddisplay");

if ( !old && old !== "none" )

jQuery.data(this[i], "olddisplay", jQuery.css(this[i], "display"));

This makes no sense to me, given how the "olddisplay" data is used. In particular, note that '&& old !== "none"' is redundant, since its a sure thing if !old is true (!)

I think the above should be:

var old = jQuery.css(this[i], "display");

if ( old && old !== "none" )

jQuery.data(this[i], "olddisplay", old);

The existing code causes a rather nasty issue when hiding and showing tr elements if you have a plugin on the page in FF3.5

see http://maps.myosotissp.com/bugs/show_hide.html

Click the "Show TR" button. Calls show() on a tr element that has been previously hidden. Because of the above bug, it resorts to setting up "elemdisplay" for a tr, and FF doesnt seem to like removing a tr thats been appended to the body element.

I may be missing something, but it looks to me like the "elemdisplay" code should never be used - and is only "needed" because of the bug in hide().