Bug Tracker

Opened 11 years ago

Closed 9 years ago

#2914 closed enhancement (invalid)

Draggable: make revert a method

Reported by: ivan Owned by: paul
Priority: minor Milestone: 1.2.4
Component: ui Version: 1.2.3
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 11 years ago.

Download all attachments as: .zip

Change History (2)

Changed 11 years ago by ivan

Attachment: ui.draggable.js.patch added

comment:1 Changed 9 years ago by dmethvin

Resolution: invalid
Status: newclosed

This should be filed in the UI bug tracker if still relevant.

http://dev.jqueryui.com

Note: See TracTickets for help on using tickets.