This minor change fixes an error when using draggable revert and grid.
Since upgrading of jQuery UI probably won't happen in the near future [1]
I looked for another way to fix the issue.
See [2]. Unlike stated in the second comment adding the lines below
won't fix the bug. Moving the lines up fixed the issue.
See also the description of the bug in [3].
[1] https://phabricator.wikimedia.org/T71386
[2] https://bugs.jqueryui.com/ticket/4696
[3] https://phabricator.wikimedia.org/T140965#
2944610
Bug: T140965
Change-Id: Ib0f386c6a2447740f44591be70c34cf4a83b88c4
});
//Generate the original position
- this.originalPosition = this.position = this._generatePosition(event);
this.originalPageX = event.pageX;
this.originalPageY = event.pageY;
+ this.originalPosition = this.position = this._generatePosition(event);
+ // These lines where moved up to fix an issue with with draggable, revert and grid
+ // See: https://bugs.jqueryui.com/ticket/4696 and https://gerrit.wikimedia.org/r/#/c/333224
+ // this.originalPageX = event.pageX;
+ // this.originalPageY = event.pageY;
//Adjust the mouse offset relative to the helper if 'cursorAt' is supplied
(o.cursorAt && this._adjustOffsetFromHelper(o.cursorAt));