Skip to main content

Search and Top Navigation

#3556 closed bug (fixed)

Opened November 08, 2008 11:54AM UTC

Closed January 08, 2009 02:04PM UTC

Draggable connectToSortable revert bug

Reported by: Flat Stanley Owned by: paul
Priority: minor Milestone: 1.7
Component: ui.draggable Version: 1.5.2
Keywords: connectToSortable revert Cc:
Blocked by: Blocking:

When we set a revert: "invalid" to a draggable connected to a sortable, we would expect that dropping the draggable onto the sortable would be considered a "valid" drop. However, I find that the item reverts to its original position and then adds itself to the sortable list. So, it seems to be considered an "invalid" drop.

Attachments (1)
  • sortable-3556.patch (1.0 KB) - added by alan.castro January 07, 2009 10:03PM UTC.

    patch that fixes revert problem with draggable that has connectToSortable option.

Change History (6)

Changed December 23, 2008 04:28PM UTC by Marc Diethelm comment:1

I found this to be fixed in trunk. Should be released in 1.6rc3. (I'm not on the team.)

Changed January 02, 2009 04:37AM UTC by scottgonzalez comment:2

milestone: TBD1.6
resolution: → fixed
status: newclosed

Closing as fixed based on Marc's comment. Feel free to re-open this ticket if you continue to have problems with 1.6rc3+.

Changed January 03, 2009 05:09AM UTC by Marc Diethelm comment:3

tested with 1.6rc4

The bug is still valid unfortunately. Maybe I can make a testcase tomorrow.

  • Trac tells me I don't have permission to reopen. *sigh*

Changed January 03, 2009 01:35PM UTC by rdworth comment:4

resolution: fixed
status: closedreopened

Changed January 07, 2009 10:02PM UTC by alan.castro comment:5

I'm attaching a patch that fixes this problem. I used the trunk version.

Changed January 08, 2009 02:04PM UTC by paul comment:6

resolution: → fixed
status: reopenedclosed

Was a bit more difficult to fix than the provided patch (the provided patch simply kills revert if connectToSortable is used). Fixed in r1557.