#2343 closed bug (fixed)

Opened February 14, 2008 12:53AM UTC

Closed March 31, 2008 02:20AM UTC

Last modified February 26, 2009 11:29AM UTC

[ui.draggable] when i try to move draggable element in "revert" mode on fly return, then flying clone attach to page, break, and loop errors. crash.

Reported by: netdog Owned by: eduardo
Priority: major Milestone: 1.5
Component: ui.core Version: 1.2.3
Keywords: revert draggable drag Cc:
Blocked by: Blocking:
Description

When i try to move draggable element in "revert" mode on fly return, then flying clone attach to page, break, and loop errors. crash.

JavaScript Log: 


Loop : 
[Break on this error]  if(ui.options.helper != 'original') self.helper.remove();
ui.draggable.ext.... (line 48)
self.helper has no properties
[Break on this error] if(ui.options.helper != 'original') self.helper.remove();
ui.draggable.ext.... (line 48)
self.helper has no properties
[Break on this error] if(ui.options.helper != 'original') self.helper.remove();
...
...
Attachments (0)
Change History (10)

Changed February 19, 2008 04:08AM UTC by netdog comment:1

You can view this bug here (in FF): http://ca.youtube.com/watch?v=ANxhIDcjpww

Changed February 19, 2008 12:42PM UTC by paul comment:2

I cannot reproduce this issue shown in the video. What version of UI are you using? Make sure you use the latest 1.5b.

Changed February 19, 2008 03:24PM UTC by netdog comment:3

I use lastest jQuery and lastest ui 1.5b.

As I understand this bag appears when one of reverted objects is destructed when there is a holded object on cursor.

set revert:3000 for slower movement of revertment objects. it will help to catch the bag.

If you cannot again reproduce this bug, I can record new video. :)

Changed February 19, 2008 06:55PM UTC by netdog comment:4

Note: Bug also produce at UI rev.4783

Changed February 20, 2008 03:34AM UTC by netdog comment:5

I think error in helper destructor. I have tried to use $(this).remove(); instead of "self.helper.remove();" and it fix the bug IMHO :)

$(ui.helper).animate(
				{ 	left: self.originalPosition.left, 		top: self.originalPosition.top } , parseInt(ui.options.revert, 10) || 500,
						function() 		{
							if(ui.options.helper != 'original') {								
								$(this).remove();
								//self.helper.remove();
								//self.clear();

								//console.log('remove');
								//console.log(this);
							}

						}

Changed February 28, 2008 10:57AM UTC by paul comment:6

owner: paul → rworth

Richard, could you take a look at this? Maybe I'm overseeing something obvious..

Changed March 25, 2008 03:15PM UTC by paul comment:7

owner: rworth → braeker

Changed March 31, 2008 02:20AM UTC by braeker comment:8

resolution: → fixed
status: new → closed

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

milestone: 1.2.4

Milestone 1.2.4 deleted

Changed February 26, 2009 11:29AM UTC by paul comment:10

milestone: → 1.5