Bug Tracker

Modify

Ticket #8629 (closed bug: worksforme)

Opened 2 years ago

Last modified 2 years ago

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:
Blocking: Blocked by:

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

comment:1 Changed 2 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 2 years ago by dmethvin

  • Owner set to jostster
  • Status changed from new to pending

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

comment:3 Changed 2 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 2 years ago by john

  • Owner jostster deleted
  • Priority changed from undecided to blocker
  • Status changed from pending to new
  • Component changed from unfiled to effects
  • Milestone changed from 1.next to 1.6

comment:5 Changed 2 years ago by dmethvin

  • Status changed from new to open

comment:6 Changed 2 years ago by jostster

Ty cowboy for the test case

comment:7 follow-up: ↓ 9 Changed 2 years ago by john

  • Priority changed from blocker to low
  • Status changed from open to closed
  • Resolution set to worksforme
  • Milestone 1.6 deleted

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 2 years ago by jostster

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

comment:9 in reply to: ↑ 7 Changed 2 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 2 years ago by anon

A perm fix would be nice john

Please follow the  bug reporting guidlines and use  jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

View

Add a comment

Modify Ticket

Action
as closed
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.