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 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.3 → 1.1.4 |
resolution: | → worksforme |
status: | new → closed |
version: | 1.1.2 → 1.1.3 |
Changed December 08, 2007 06:14PM UTC by comment:2
resolution: | worksforme |
---|---|
status: | closed → reopened |
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 comment:3
resolution: | → fixed |
---|---|
status: | reopened → closed |
Right, you can't slideUp table rows (since table rows don't have any height).
Changed October 14, 2010 05:35PM UTC by 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 comment:5
nevermind, this is totally closed :)
Works for me:
http://dev.jquery.com/~john/ticket/1082/