Ticket #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: | ||
| Blocking: | Blocked by: |
Description (last modified by john) (diff)
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
comment:1 Changed 6 years ago by john
- Status changed from new to closed
- Version changed from 1.1.2 to 1.1.3
- Resolution set to worksforme
- Description modified (diff)
- Milestone changed from 1.1.3 to 1.1.4
comment:2 Changed 5 years ago by nevyn
- Status changed from closed to reopened
- Resolution worksforme deleted
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 5 years ago by john
- Status changed from reopened to closed
- Resolution set to fixed
Right, you can't slideUp table rows (since table rows don't have any height).
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

Works for me: http://dev.jquery.com/~john/ticket/1082/