Opened 13 years ago

Closed 12 years ago

Last modified 12 years ago

#5882 closed bug (fixed)

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

Change History (4)

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

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

comment:2 Changed 12 years ago by Scott González

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

comment:3 Changed 12 years ago by Scott González

Milestone: TBD1.8.9

comment:4 Changed 12 years ago by Scott González

#5055 is a duplicate of this ticket.

Note: See TracTickets for help on using tickets.