Skip to main content

Search and Top Navigation

#5882 closed bug (fixed)

Opened July 30, 2010 11:47AM UTC

Closed January 19, 2011 03:46PM UTC

Last modified January 19, 2011 03:53PM UTC

UI.Sortable .sortable( "cancel" ) call the this._mouseUp method without param

Reported by: tobias.istvan Owned by:
Priority: minor Milestone: 1.8.9
Component: ui.sortable Version: 1.8.2
Keywords: sortable cancel Cc:
Blocked by: Blocking:
Description

The .sortable( "cancel" ) call the this._mouseUp without param. For the _mouseUp method require this param.

Also:

  • not check the existing of the this.placeholder in the condition
  • not check the existing of the this.domPosition in the condition
Attachments (0)
Change History (4)

Changed January 19, 2011 03:46PM UTC by Scott González comment:1

resolution: → fixed
status: newclosed

Sortable: pass a fake event object to _mouseUp on cancel. Fixes #5882 - UI.Sortable .sortable( cancel ) call the this._mouseUp method without param.

(cherry picked from commit cf0c32f335479d00224d69f3c5ff53d147a6a8e6)

Changeset: 465c4c2cbef33fc4491d862245180b6441875c15

Changed January 19, 2011 03:46PM UTC by Scott González comment:2

Sortable: pass a fake event object to _mouseUp on cancel. Fixes #5882 - UI.Sortable .sortable( cancel ) call the this._mouseUp method without param.

Changeset: cf0c32f335479d00224d69f3c5ff53d147a6a8e6

Changed January 19, 2011 03:46PM UTC by scottgonzalez comment:3

milestone: TBD1.8.9

Changed January 19, 2011 03:53PM UTC by scottgonzalez comment:4

#5055 is a duplicate of this ticket.