In IE6, changes to checkbox settings lost when the containing <li> dragged

Problem traced to Sortable.onHover:

  dropon.parentNode.insertBefore(element, xxx);

where xxx depends on the direction of drag: up = dropon (overlap > 0.5), down = nextElement.

Alerting element.innerHTML before and after shows IE reverting to original code, however FF shows the original regardless (but still works). Bug does not occur in Opera 8.51 either (Safari unknown).

Proposed patch: add var and check to onHover:

    onHover:function(element, dropon, overlap) {
      var oldParentNode, elHTML=element.innerHTML;
      .
      .
      . // rest of function
      .
      .
      if (element.innerHTML !== elHTML){
        element.innerHTML = elHTML;
      }
    },

Proposed patch seems to make the element non-draggable once moved around in IE.

hey, i think ive got a solution to this bug. basically on the startDrag event, you get all the checkbox's in the element you are dragging and put them in an array along with its check state then on the updateDrag event you update the values of each check box. here is what it looks like:

cbstate: [],

getElementsByType: function(element,type){

var ele = Form.getElements(element); var returnArr = new Array(); ele.each(function(e){if(e.type==type)returnArr.push(e)}); return returnArr;

},

startDrag: function(event) {

this.dragging = true;

cbstate = new Array(); this.getElementsByType(this.element, 'checkbox').each(function(e){cbstate.push({cb:e,s:e.checked})});

. . . // rest of function . .

},

updateDrag: function(event, pointer) {

if(!this.dragging) this.startDrag(event);

cbstate.each(function(e){e.cb.checked = e.s;});

. . . // rest of function . .

},

i have tested this in ie & firefox and seems to work fine. hope this helps, jamie

The fix works for me.

Updating function "updateDrag" did not work for me so i updated function "finishDrag" and it works just fine