Bug Tracker

Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#8629 closed bug (worksforme)

fadeIn for a table row does not remove display: block

Reported by: jostster Owned by:
Priority: low Milestone:
Component: effects Version: 1.5.1
Keywords: Cc:
Blocked by: Blocking:

Description

This is a duplicate of http://bugs.jquery.com/ticket/2185 but still is not fixed in 1.5.1. I have a table row with the only style being

<tr style="border: thin solid lightgrey;"></tr> 

I then clone that row and do a fadeIn but it adds display: block to the end so it looks like this

<tr style="border: thin solid lightgrey; display: block;"></tr> 

In firefox this makes the table row look EXTREMELY ugly. In safari it looks normal.

I have created a JSFiddle page with parts of the table and jquery source code to duplicate the issue

http://jsfiddle.net/P2MX8/

Change History (10)

comment:1 Changed 8 years ago by jostster

I forgot to mention in the last table row for each section click the + sign to duplicate this issue.

comment:2 Changed 8 years ago by dmethvin

Owner: set to jostster
Status: newpending

Can you create a reduced test case? Your test case is too complex to clearly demonstrate a bug in core.

comment:3 Changed 8 years ago by cowboy

Here's a simplified test case that shows that the faded-in TR gets display: block in Firefox 2-4.

http://jsfiddle.net/cowboy/DzH9x/

comment:4 Changed 8 years ago by john

Component: unfiledeffects
Milestone: 1.next1.6
Owner: jostster deleted
Priority: undecidedblocker
Status: pendingnew

comment:5 Changed 8 years ago by dmethvin

Status: newopen

comment:6 Changed 8 years ago by jostster

Ty cowboy for the test case

comment:7 Changed 8 years ago by john

Milestone: 1.6
Priority: blockerlow
Resolution: worksforme
Status: openclosed

The fix for this ends up being rather easy: Just .hide() the element after you insert it into the page: http://jsfiddle.net/DzH9x/5/

comment:8 Changed 8 years ago by jostster

Last edited 8 years ago by jostster (previous) (diff)

comment:9 in reply to:  7 Changed 8 years ago by jostster

Replying to john:

The fix for this ends up being rather easy: Just .hide() the element after you insert it into the page: http://jsfiddle.net/DzH9x/5/

Providing a work around shouldn't be a permanent fix...

comment:10 Changed 8 years ago by anon

A perm fix would be nice john

Note: See TracTickets for help on using tickets.