Skip to main content

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!)

Attachments (1)
  • draggable-revert.patch (2.3 KB) - added by kevingessner August 12, 2008 10:11AM UTC.

    Patch for ui.draggable adding callback capability to revert: (with test)

Change History (2)

Changed August 16, 2008 08:26PM UTC by Cloudream comment:1

milestone: → 1.6

Changed September 18, 2008 09:51AM UTC by paul comment:2

resolution: → fixed
status: newclosed

Thanks, implemented in r689.