Opened 12 years ago
Closed 12 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 12 years ago by
| Attachment: | ui.draggable.js.patch added |
|---|
comment:1 Changed 12 years ago by
| Milestone: | 1.2.4 |
|---|
comment:2 Changed 12 years ago by
| Component: | ui.core → ui.draggable |
|---|---|
| Version: | 1.2.3 → 1.5b4 |
comment:3 Changed 12 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