defaultDisplay returns block instead of table-row for a tr in FF

[[email protected]…]

​http://jsfiddle.net/mh8mF/

.toggle(true/false) on <tr/> may sometimes not behave properly on table with "colspan" See fiddle example, the row on "Bad|Table" get collapsed.

Using .hide()/.show() will also trigger the same problem

[timmywil]

Valid html needs to be appended to the table. See ​http://jsfiddle.net/timmywil/mh8mF/4/

[[email protected]…]

The original post is appended to the table.

The two "tables", one is good (expected result) and one is bad (buggy). The different is due to the usage of toggle() function.

[timmywil]

Ok, I read that wrong. But it actually worked for me in Chrome and IE6. It seems this is only an issue in Firefox.

[timmywil]

I've only checked FF 7 so far: ​http://jsfiddle.net/timmywil/mh8mF/9/

[Rick Waldron]

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

More Details:

• ​https://github.com/jquery/jquery/pull/530
• ​http://bugs.jquery.com/ticket/10416 Changeset: 22f2e8b3dc18dede5f1ccb28cbdf8cb5bcde115f

[timmywil]

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

This reverts commit 22f2e8b3dc18dede5f1ccb28cbdf8cb5bcde115f.

Changeset: 0752687612d190f608e64181148ced1a4adfa5d6

[Rick Waldron]

#10741 is a duplicate of this ticket.

[[email protected]…]

​http://jsfiddle.net/davidtong/gxNeD/

Try it in Chrome/IE and then in FF. The red console div should be empty in Chrome/IE, but in FF it says 'block'

My workaround:

var old_display = j_div.css('display'); cache custom display value (e.g. display:table-cell)

j_div.detach().hide(); upon detach, j_div.css('display') returns 'block' even if we set display:table-cell in the CSS

$j._data(j_div[0], "olddisplay", old_display);

[Rick Waldron]

Which version of FF are you testing in?

[Rick Waldron]

To note, the original fix for this bug caused a regression and we plan on revisiting the issue during the 1.7 lifecycle.

[davidalism]

re: rwaldron

Tested on FF 3.6 and FF 7. They both report "block". Chrome and IE return "" (empty string).

[mikesherov]

#10918 is a duplicate of this ticket.

[mikesherov]

fully reduced test case: ​http://jsfiddle.net/73EZY/ confirmed still present in FF12

[gibson042]

Is this a job for wrapMap?

[mikesherov]

This looks like a bug in FF itself. Gibson042, can you confirm?

[gibson042]

If this is a bug, it's easy to work around. Check out what happens when you detach the entire table: ​http://jsfiddle.net/73EZY/1/

[anonymous]

Replying to gibson042:

If this is a bug, it's easy to work around. Check out what happens when you detach the entire table: ​http://jsfiddle.net/73EZY/1/

This example doesn't seems related to the original bug. This is just an example of usage, which doesn't even hit the bug situation.

[gibson042]

#12272 is a duplicate of this ticket.

[Oleg]

Fix #10416. Don't trust computed styles on detached elements. Close gh-941.

Changeset: bea5ecbba72f22e8924186d9b81839122ec86aef