Ticket #6950 (closed bug: fixed)

Opened 4 years ago

Last modified 3 years ago

ie 8 invalid drag and drop error

Reported by: elpy Owned by:
Priority: minor Milestone: 1.8.14
Component: ui.draggable Version: 1.8.9
Keywords: Cc:
Blocking: Blocked by:

Description

 http://jsfiddle.net/mmC5L/2/

IE 8 returns illegal char error on line 1148 in jquery-ui-1.8.9.custom.js; the script works in firefox 3.6.13 and safari 4 public beta (WINDOWS version) but not IE8 ( i have not tested this in other versions than the ones i listed)

this is that line:

if(!this.options.axis || this.options.axis != "y") this.helper[0].style.left = this.position.left+'px';

the error occurs when the "axis" option is not set and the "grid" option (either x or y) is set to 0, like i did here:

$( ".dragable" ).draggable({ grid: [ 0,12 ], stack: "placeHolder div", distance: 24, containment: 'parent' });

a workarround is to supply the axis option, but i think a check should be added to prevent this error.

Change History

comment:1 Changed 3 years ago by kborchers

  • Status changed from new to closed
  • Resolution set to fixed

Draggable: Check for grid elements set to 0 to prevent divide by 0 error causing invalid argument errors in IE. Fixed #6950 - ie 8 invalid drag and drop error

Changeset: d136e4333e117b33aa16eb9a3a5831a25b1a2dd9

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

Merge pull request #268 from kborchers/bug_6950

Draggable: Check for grid elements set to 0 to prevent divide by 0 error causing invalid argument errors in IE. Fixed #6950 - ie 8 invalid drag and drop error

Changeset: aec3f1893276bbb5ace7ae6d2b340f273489160c

comment:3 Changed 3 years ago by kborchers

Draggable: Check for grid elements set to 0 to prevent divide by 0 error causing invalid argument errors in IE. Fixed #6950 - ie 8 invalid drag and drop error (cherry picked from commit d136e4333e117b33aa16eb9a3a5831a25b1a2dd9)

Changeset: 16b4ffb791f50d8852289db88eef175b6d1adb97

comment:4 Changed 3 years ago by scott.gonzalez

  • Milestone changed from 1.9 to 1.8.14

comment:5 Changed 3 years ago by scott.gonzalez

  • Type changed from enhancement to bug
Note: See TracTickets for help on using tickets.