Sortable: placeholder height/width incorrect if sortable item is resized

[nicholasstephan]

If sortable items have a css height then that height is used for the placeholder when dragging, even after that item has been resized:

​http://jsfiddle.net/nicholasstephan/e9Dk3/

Remove the height attribute, or move it to an inline style, and it behaves as you'd expect:

​http://jsfiddle.net/nicholasstephan/e9Dk3/1/

[tj.vantoll]

Probably related to #4345, need to investigate a little more.

[tj.vantoll]

This is not related to #4345 at all.

Updated and reduced test case against master: ​http://jsfiddle.net/tj_vantoll/UsdRn/. This is not just for the height but also the width. It appears the placeholder's dimensions need to be recalculated on mouse start.

[Scott González]

I'd actually say this is working as intended. Though the documentation on this isn't very good, the logic has always been that providing a custom placeholder class means that dimensions are not copied unless the placeholder ends up with no dimensions and forcePlaceholderSize is set to true. Since the placeholder has a defined height from the li rule, that's what gets used.

While not ideal, you can add a class to all of the items and then put the height on that class: ​http://jsfiddle.net/UsdRn/8/

I think any change to this is likely to break existing uses, so I'm going to close as not a bug since the current behavior was actually intended. We'll likely reconsider pretty much every aspect of this during the rewrite.