Skip to main content

Search and Top Navigation

#4314 closed bug (invalid)

Opened March 10, 2009 08:25AM UTC

Closed November 12, 2012 09:10AM UTC

Sortable: DOM height() not calculated properly after updating, while dragging

Reported by: tehmeph Owned by: tehmeph
Priority: major Milestone: 2.0.0
Component: ui.sortable Version: 1.7
Keywords: Cc:
Blocked by: Blocking:

I've got two connected lists of different widths. One of the items I'm dragging between the two of them has two floated child elements, which, upon dragging out of one sortable list into another, resize, vertically and horizontally.

I'm using a custom animation callback to change the width of the element, which, because of the floats, changes the height as well. I have, however, debugged the output for ui.item.height(), ui.placeholder.height(), and ui.helper.height(), and for one of the widgets the pre-drag height is 133px; after dragging into the new list, and it is visually taller (and thinner), the height calculated by .height() is still 133px and it is not 189px (as it should be) until I begin draggin again.

The end result of this is that some widgets with greater variation in size - one "grows" by as much as 200px - makes the sortable list nearly unusable because of the massive chunk of whitespace between the actively sorted element and it's neighbors.

Source attached.

Attachments (1)
  • jqbug (3.1 KB) - added by tehmeph March 10, 2009 08:25AM UTC.
Change History (8)

Changed May 07, 2009 10:57AM UTC by jzaefferer comment:1

milestone: TBD1.7.2

Changed May 07, 2009 01:22PM UTC by jzaefferer comment:2


Changed August 27, 2009 07:38PM UTC by zertox comment:3

Same issue here. After changing the width of the helper opject, the height stays the same.

Changed October 11, 2012 02:54PM UTC by scottgonzalez comment:4


Changed October 25, 2012 01:10AM UTC by petersendidit comment:5

owner: → tehmeph
status: newpending
summary: DOM height() not calculated properly after updating, while draggingSortable: DOM height() not calculated properly after updating, while dragging

Thanks for taking the time to contribute to the jQuery UI project! Please provide a complete reduced test case on jsFiddle to help us assess your ticket.

Additionally, be sure to test against the git version of both jQuery UI and jQuery to ensure the issue still exists. To get you started, use this boilerplate: the link and click to "Fork" (in the top menu) to get started.

Changed October 25, 2012 02:30AM UTC by mikesherov comment:6

owner: tehmeph
status: pendingnew

petersendidit, can we try making a test case from the attachment instead of pending this one?

Changed October 29, 2012 05:27AM UTC by mikesherov comment:7

owner: → tehmeph
status: newpending

petersendidit, you were right!

Changed November 12, 2012 09:10AM UTC by trac-o-bot comment:8

resolution: → invalid
status: pendingclosed

Because we get so many tickets, we often need to return them to the initial reporter for more information. If that person does not reply within 14 days, the ticket will automatically be closed, and that has happened in this case. If you still are interested in pursuing this issue, feel free to add a comment with the requested information and we will be happy to reopen the ticket if it is still valid. Thanks!