Opened 8 years ago

Closed 8 years ago

Last modified 7 years ago

#4657 closed bug (fixed)

Update docs, revert supports a number parameter for animation speed

Reported by: crankharder Owned by:
Priority: major Milestone: 1.8
Component: ui.sortable Version: 1.7.2
Keywords: revert Cc:
Blocked by: Blocking:


Should be self-explanatory:

I'd like to be able to control how fast an option reverts back to its position.

Should act like other jquery speed settings, slow, normal, fast (i think?) or an integer which represents milliseconds.

Change History (4)

comment:1 Changed 8 years ago by crankharder

I tracked this down to the animate() function inside of sortable here:


left: cur.left - this.offset.parent.left - self.margins.left + (this.offsetParent[0] == document.body ? 0 : this.offsetParent[0].scrollLeft), top: - - + (this.offsetParent[0] == document.body ? 0 : this.offsetParent[0].scrollTop)

}, parseInt(this.options.revert, 10)
500, function() {



The second param is: parseInt(this.options.revert, 10)
500, which is the duration of the animation... I'm not sure what the parseInt part is trying to do, but if I replace that whole thing with a custom speed then I get the result I want. I don't think this can be set by some external option as of right now -- maybe it can??

comment:2 Changed 8 years ago by crankharder

Okay -- the docs explicitly state that revert takes a boolean, and the examples show true/false being passed in:

Based on the code above, taken from 1.7.2 that revert actually will take a number and convert that to the animation speed.

Passing revert:100 as an option confirms it as well.

I've no idea if tests support this, but it seems to work in practice... Maybe somebody should update the docs to reflect that?

comment:3 Changed 8 years ago by Jörn Zaefferer

Milestone: TBD1.8
Priority: minormajor
Summary: new sortable option: revertSpeedUpdate docs, revert supports a number parameter for animation speed
Type: enhancementbug
Version: 1.7.2

Docs need to be updated.

comment:4 Changed 8 years ago by paul

Resolution: fixed
Status: newclosed


Note: See TracTickets for help on using tickets.