Opened 9 years ago

Closed 9 years ago

#2914 closed enhancement (fixed)

Draggable: make revert a method

Reported by: ivan Owned by: paul
Priority: minor Milestone: 1.5.2
Component: ui.draggable Version: 1.5b4
Keywords: Cc:
Blocked by: Blocking:

Description

A pretty common need with draggables/droppables is to accept the drop, do some processing, and then, if some conditions are not met, restore the draggable to the original position.
I think it would make sense to make 'revert' callable externally (for example in the 'drop' method of the droppable).
The attached patch does that: unfortunately, at the moment it requires setting draggableInstance.cancelHelperRemoval = true; in the draggable 'stop' or in the droppable 'drop' methods, to prevent the helper from being removed too soon (see also here for a discussion).

Attachments (1)

ui.draggable.js.patch (1.8 KB) - added by ivan 9 years ago.

Download all attachments as: .zip

Change History (4)

Changed 9 years ago by ivan

Attachment: ui.draggable.js.patch added

comment:1 Changed 9 years ago by (none)

Milestone: 1.2.4

Milestone 1.2.4 deleted

comment:2 Changed 9 years ago by paul

Component: ui.coreui.draggable
Version: 1.2.31.5b4

comment:3 Changed 9 years ago by paul

Milestone: 1.5.2
Resolution: fixed
Status: newclosed

Done in r402. Use revert: "invalid"

Note: See TracTickets for help on using tickets.