Opened 15 years ago
Closed 15 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)
Change History (4)
Changed 15 years ago by
Attachment: | ui.draggable.js.patch added |
---|
comment:1 Changed 15 years ago by
Milestone: | 1.2.4 |
---|
comment:2 Changed 15 years ago by
Component: | ui.core → ui.draggable |
---|---|
Version: | 1.2.3 → 1.5b4 |
comment:3 Changed 15 years ago by
Milestone: | → 1.5.2 |
---|---|
Resolution: | → fixed |
Status: | new → closed |
Done in r402. Use revert: "invalid"
Note: See
TracTickets for help on using
tickets.
Milestone 1.2.4 deleted