Minor Issues in Row Reordering Sample... #306

Closed
tmcconechy opened this Issue Mar 20, 2012 · 0 comments

Comments

Projects
None yet
2 participants

Two Small issues.. (well one might be considered a dom leak).

A)

In the current example9-row-reordering.html

  1. Drag and row to reorder...
  2. Look at the -> grid-canvas dom element -> notice empty div 's elements in there.
  3. the more elements you move the more div 's end up in there. So this code be symptom of a larger issue?

Might want to look into this but one cheap fix is to add this code:

grid.render();
grid.render;
....
$('#myGrid .grid-canvas > div:empty').filter(function (index) { 
    return $(this).children().length < 1; 
}).remove();

B) In the current example9-row-reordering.html

  1. Drag a row to the cycle bin..
  2. Release it at the moment the proxy element touches it.. Since this object is very long this could be confusing.

You can fix this by using the option from the dnd plug. To force it to use the mode of mouse is really over it...

$("#dropzone").
     bind("dropstart", function (e, dd) {
   ......
 });
 $.drop({ mode:true });

mleibman closed this in 2ae0b4f Apr 13, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment