Opened 11 years ago

Closed 11 years ago

Last modified 10 years ago

#3666 closed bug (fixed)

connectToSortable breaks if argument is array

Reported by: Marc Diethelm Owned by:
Priority: major Milestone: 1.7
Component: ui.draggable Version: 1.6rc2
Keywords: draggable, sortable, connectToSortable, array Cc:
Blocked by: Blocking:

Description

A draggable can be linked to a sortable in two ways: by setting 'connectToSortable' to a selector or to an array of selectors. Using an array fails to provide the expected functionality however, the sortable does not 'accept' the dragged item.

Attachments (3)

sortable.patch (1.2 KB) - added by alan.castro 11 years ago.
small change in the patch
sortable-bug-3666.patch (736 bytes) - added by alan.castro 11 years ago.
better patch than the last one
draggable_sortable.test.html (1.5 KB) - added by Marc Diethelm 11 years ago.
Testcase

Download all attachments as: .zip

Change History (10)

comment:1 Changed 11 years ago by Scott González

Milestone: TBD1.next

comment:2 Changed 11 years ago by Marc Diethelm

In 1.6rc4 the behavior is reversed. Now a single selector string produces no result while an array of selectors works. Filing a new bug #3739.

comment:3 Changed 11 years ago by alan.castro

I'm attaching a patch that fixes this issue, and by testing it i realized another bug: #3787

comment:4 Changed 11 years ago by alan.castro

ps: i used jquery ui trunk version.

Changed 11 years ago by alan.castro

Attachment: sortable.patch added

small change in the patch

Changed 11 years ago by alan.castro

Attachment: sortable-bug-3666.patch added

better patch than the last one

comment:5 Changed 11 years ago by Marc Diethelm

After #3984 fix were back were we started: array is not accepted.

Changed 11 years ago by Marc Diethelm

Testcase

comment:6 Changed 11 years ago by Marc Diethelm

Feb 12th: Array works again, String fails. This is better. However the documentation states:
"Specifying this option as an array of selectors is deprecated." This needs to be changed.

comment:7 Changed 11 years ago by Scott González

Milestone: 1.next1.6
Resolution: fixed
Status: newclosed

Support for arrays has been removed. You must specify a selector now. I've updated the docs. Thanks.

Note: See TracTickets for help on using tickets.