Skip to main content

Search and Top Navigation

#2718 closed bug (worksforme)

Opened April 19, 2008 02:58AM UTC

Closed April 30, 2008 10:37AM UTC

Last modified May 24, 2008 03:39AM UTC

draggable connectToSortable causes "no properties" error

Reported by: DanSherman Owned by: paul
Priority: major Milestone:
Component: ui.core Version: 1.2.3
Keywords: Cc:
Blocked by: Blocking:
Description

Using jquery-1.2.4a.js & jquery.ui-all-1.5b2.js the code bellow causes a "inst.placeholder has no properties" error on execution of this line. inst.placeholder.remove();

The issue accurs when a dragable div is selected from div.f_obj_temp an hovered over div.f_page, but not dropped. As soon as the draggable is moved outside the bounds of

div.f_page the error occurs. when the helper is finally dropped, it holds it's positions and is not selectable for future dragging. The last part can be worked around by manually removing it inside the sortable stop function.

one possible improvement i can think of would be to allow connection to multiple sortables so that the draggable could be dropped in any of the locations.

If need be please let me know, and i can provide a working example of the issue.

$("#temps > .f_obj_temp").draggable({

helper: 'clone',

opacity: 0.8,

cursorAt: [0,0],

connectToSortable: '.f_page',

containment: '#the_main_table',

appendTo: 'body'

});

$('#inner_form > .f_page').sortable({

containment: 'parent',

appendTo: 'body',

axis: 'y',

opacity: 0.8,

start: function(ev,ui){

$('#trash').empty();

$('#trash').html("Started");

}

});

-Dan S

Attachments (0)
Change History (3)

Changed April 23, 2008 09:51AM UTC by paul comment:1

status: newassigned

Changed April 30, 2008 10:37AM UTC by paul comment:2

resolution: → worksforme
status: assignedclosed

I cannot reproduce your issue. There is a test file that I would ask you to use and see if it works - it's at trunk/ui/tests/sortable_draggable.html in SVN. Thanks!

Changed May 24, 2008 03:39AM UTC by comment:3

milestone: 1.2.4

Milestone 1.2.4 deleted