Bug Tracker

Opened 13 years ago

Closed 12 years ago

Last modified 8 years ago

#1082 closed bug (fixed)

removing table row after effect does not free space

Reported by: jackfrost86 Owned by:
Priority: minor Milestone: 1.1.4
Component: effects Version: 1.1.3
Keywords: Cc:
Blocked by: Blocking:

Description (last modified by john)

removing a tr after an effect with code such as this: <tr id="product1">...</tr> ... $("tr#product1").fadeOut("slow", function() {

$("tr#product1").remove();

});

does not work properly in Firefox, although it does work properly in Safari and IE. The table row fades out and is removed from the DOM but the space it occupied remains used; the rows below it do not slide up. Simply remove()-ing the object does work as expected: the space the table row used is released.

See http://jquery.com/pipermail/discuss_jquery.com/2006-November/016579.html for more information.

Change History (5)

comment:1 Changed 13 years ago by john

Description: modified (diff)
Milestone: 1.1.31.1.4
Resolution: worksforme
Status: newclosed
Version: 1.1.21.1.3

comment:2 Changed 12 years ago by nevyn

Resolution: worksforme
Status: closedreopened

I get this error as well; http://nevyn.nu/1082/

It works with fadeOut, but not with slideUp. Also, applying the effect to a tr instead of each individual td seems to mess up the layout, but I suppose that's why did apply it to the tds...

comment:3 Changed 12 years ago by john

Resolution: fixed
Status: reopenedclosed

Right, you can't slideUp table rows (since table rows don't have any height).

comment:4 Changed 9 years ago by anonymous

this should not be closed. regardless of animation methods, the remove method only empties out a table row, and a simple count of the row elements will prove this. test case coming soon.

comment:5 Changed 9 years ago by anonymous

nevermind, this is totally closed :)

Note: See TracTickets for help on using tickets.