Opened 14 years ago

Closed 11 years ago

#4482 closed bug (patcheswelcome)

Sortable: Placeholder height not being calculated/applied properly in IE7/6

Reported by: bbonifield Owned by:
Priority: major Milestone: 2.0.0
Component: ui.sortable Version: 1.7.1
Keywords: haspatch Cc:
Blocked by: Blocking:

Description

When using a sortable in IE7/6, the height of the placeholder element is not inheriting the height of the current target draggable element. UI tries to add inline CSS for this purpose, but it seems to be failing in IE < 8.

REF: http://jsbin.com/obogo Example source also attached

Attachments (1)

sortableTest.html (1.1 KB) - added by bbonifield 14 years ago.

Download all attachments as: .zip

Change History (7)

Changed 14 years ago by bbonifield

Attachment: sortableTest.html added

comment:1 Changed 14 years ago by Jörn Zaefferer

Milestone: TBD1.7.2

comment:2 Changed 14 years ago by Jörn Zaefferer

Milestone: 1.7.21.8

comment:3 Changed 13 years ago by peter4u

workaround:

start: function (event, ui) {
    ui.placeholder.height(ui.helper.height());
}

comment:4 Changed 11 years ago by Scott González

Milestone: 1.9.02.0.0

comment:5 Changed 11 years ago by mikesherov

Keywords: haspatch added
Status: newopen

confirmed in IE7 on latest: http://jsbin.com/obogo/14/edit

comment:6 Changed 11 years ago by tj.vantoll

Resolution: patcheswelcome
Status: openclosed

I still see the issue in IE7 against 1.10.0 - http://jsbin.com/obogo/15/edit. It looks like as the OP stated 4 years ago the inline style is being added but it not taking effect in IE7 for whatever reason, but works fine in IE8+.

At this point, we're not going to invest any more time into searching for a workaround that we're comfortable implementing. If anyone has a proposed fix, we'll gladly review a pull request.

Note: See TracTickets for help on using tickets.