Ticket #2122 (closed bug: fixed)
Droppable/Draggable element comparison needs to be changed on trunk.
| Reported by: | mnichols | Owned by: | paul |
|---|---|---|---|
| Priority: | major | Milestone: | 1.2.2 |
| Component: | ui | Version: | 1.2.1 |
| Keywords: | droppable trunk | Cc: | |
| Blocking: | Blocked by: |
Description
Since the rewrite on droppable.js in the trunk, the comparisons within the droppable methods over,out, and drop need to be changed . The component now lets you drop a draggable that is the same element. Please see attached patch. Also note that this patch contains the fixes in #2063 where the ui object is passed to the accept function on droppable.
Attachments
Change History
Changed 5 years ago by mnichols
-
attachment
uidroppable-fix_element_comparer_and_accept_fn.patch
added
comment:1 Changed 5 years ago by mnichols
note that using the patch I attached here, the following is possible in the droppable's accept function:
function tryAccept(ui){
console.debug('draggable:' + ui.draggable); console.debug('droppable:' + ui.instance); console.debug('droppable element:' + $j(ui.instance.element).attr('id')); console.debug('draggable element:' + $j(ui.element).attr('id'));
return true;
}
This let's the developer do runtime checks on the target droppable to see if an draggable element should be accepted or not. The current args do not allow this easily.
Please follow the bug reporting guidlines and use jsFiddle when providing test cases and demonstrations instead of pasting the code in the ticket.

This patch fixes element comparisons on droppable in the trunk and also extends the accept function argument