Opened 8 years ago

Closed 8 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).

Change History (3)

comment:1 Changed 8 years ago by anonymous

  • Milestone 1.2.4 deleted

Milestone 1.2.4 deleted

comment:2 Changed 8 years ago by paul

  • Component changed from ui.core to ui.draggable
  • Version changed from 1.2.3 to 1.5b4

comment:3 Changed 8 years ago by paul

  • Milestone set to 1.5.2
  • Resolution set to fixed
  • Status changed from new to closed

Done in r402. Use revert: "invalid"

Note: See TracTickets for help on using tickets.