Opened 14 years ago

Closed 14 years ago

Last modified 13 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 14 years ago.
small change in the patch
sortable-bug-3666.patch​ (736 bytes) - added by alan.castro 14 years ago.
better patch than the last one
draggable_sortable.test.html​ (1.5 KB) - added by Marc Diethelm 14 years ago.
Testcase

Download all attachments as: .zip

Change History (10)

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

Milestone: TBD → 1.next

comment:2 Changed 14 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 14 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 14 years ago by alan.castro

ps: i used jquery ui trunk version.

Changed 14 years ago by alan.castro

Attachment: sortable.patch​ added

small change in the patch

Changed 14 years ago by alan.castro

Attachment: sortable-bug-3666.patch​ added

better patch than the last one

comment:5 Changed 14 years ago by Marc Diethelm

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

Changed 14 years ago by Marc Diethelm

Attachment: draggable_sortable.test.html​ added

Testcase

comment:6 Changed 14 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 14 years ago by Scott González

Milestone: 1.next → 1.6
Resolution: → fixed
Status: new → closed

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.