Skip to main content

Bug Tracker

Side navigation

#1082 closed bug (fixed)

Opened March 28, 2007 09:32PM UTC

Closed December 12, 2007 08:22PM UTC

Last modified March 15, 2012 01:23AM UTC

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

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.

Attachments (0)
Change History (5)

Changed July 21, 2007 01:14AM UTC by john comment:1

description: 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.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.
milestone: 1.1.31.1.4
resolution: → worksforme
status: newclosed
version: 1.1.21.1.3

Changed December 08, 2007 06:14PM UTC by nevyn comment:2

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...

Changed December 12, 2007 08:22PM UTC by john comment:3

resolution: → fixed
status: reopenedclosed

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

Changed October 14, 2010 05:35PM UTC by anonymous comment:4

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.

Changed October 14, 2010 05:53PM UTC by anonymous comment:5

nevermind, this is totally closed :)