:visible not workgin on tr in IE 7

[pete in atlanta]

we're using a toggle edit function that changes the visibility of a table row. Here's the function:

flipEdit = function(num) {

var readId = "#read"+num;

var editId = "#edit"+num; if ($(readId).is(":visible")) $(readId).fadeOut('fast', function() {$(editId).fadeIn();}) else $(editId).fadeOut('fast', function() {$(readId).fadeIn();})

$("#example .round:visible").each(function(){alert(this.id);})

};

It works great on divs and tr's work in FF but not IE7. What happens is that you can toggle to see the edit but can't toggle back to the read only. Thanks and keep up the great work, -PIA

[dmethvin]

Can you put together a complete test case with html and script?

[hati]

change this to Sizzle.selectors.filters.hidden = function(elem){

return elem.offsetWidth === 0

elem.offsetHeight === 0;

};

Sizzle.selectors.filters.visible = function(elem){

return elem.offsetWidth > 0

elem.offsetHeight > 0;

that Sizzle.selectors.filters.hidden = function(elem){

return "hidden" === elem.type

jQuery.css(elem, "display") === "none"

jQuery.css(elem, "visibility") === "hidden";

};

Sizzle.selectors.filters.visible = function(elem){

return "hidden" !== elem.type &&

jQuery.css(elem, "display") !== "none" && jQuery.css(elem, "visibility") !== "hidden";

};

[howardr]

This has been fixed and closed in another ticket ​http://dev.jquery.com/ticket/4374

This ticket should be closed

[kjz44]

I have the same problem with :visible and TRs on IE7. I solved it by changing my selector from 'tbody tr:visible' to 'tbody tr:not(:hidden)'. Logically, both selectors should produce the same node set, but do not. I suspect the difference is due to the issue described in ticket #4374. A change was submitted (#6286) which seems to make :visible == :not(:hidden) and it looks to me like the change will make :not(:hidden) suffer from the same problem on TRs. Perhaps the solution is to write a custom filter.

[dmethvin]

Dup of #4374