Search and Top Navigation
#3175 closed enhancement (fixed)
Opened August 12, 2008 10:10AM UTC
Closed September 18, 2008 09:51AM UTC
Draggable revert: option can be a callback
Reported by: | kevingessner | Owned by: | paul |
---|---|---|---|
Priority: | minor | Milestone: | 1.7 |
Component: | ui.draggable | Version: | 1.5.2 |
Keywords: | draggable revert | Cc: | |
Blocked by: | Blocking: |
Description
Attached is a patch w/ test that adds the capability for draggable's revert: option to be a function, rather than just true/false/(in)valid. The function is called on drop, and the return value is used the same as constant true/false values. It gives more flexibility than simply using a constant value.
Caveat: I'm not sure what arguments should be passed to the callback, if any - I couldn't tell where the standard (e, ui) come from, or if those are appropriate.
(I hope I'm doing this right. Sorry, it's my first ticket/contribution here. And I'm not sure how the test suite works, so mine's a separate file. Thanks!)