Bug Tracker

Ticket #1082 (closed bug: fixed)

Opened 2 years ago

Last modified 3 months ago

removing table row after effect does not free space

Reported by: jackfrost86 Assigned to: anonymous
Type: bug Priority: minor
Milestone: 1.1.4 Component: fx
Version: 1.1.3 Keywords:
Cc: Needs: Review

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.

Attachments

Change History

Changed 1 year ago by john

  • status changed from new to closed
  • version changed from 1.1.2 to 1.1.3
  • resolution set to worksforme
  • description changed from 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. to 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 changed from 1.1.3 to 1.1.4

Changed 1 year ago by nevyn

  • status changed from closed to reopened
  • resolution 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...

Changed 1 year 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).

Note: See TracTickets for help on using tickets.