snapToGridDuringDrag interferes with undo/redo

[tsi]

Seems like setting snapToGridDuringDrag: true somehow messes with the undo stack of https://github.com/iVis-at-Bilkent/cytoscape.js-undo-redo

Here's what I get:

• selected multiple elements
• drag to move them all
• press cmd+z to undo
• only a single (the last) item moved back to previous position

This doesn't happen with snapToGridDuringDrag: false

Thanks!!

[canbax]

The real bug is not related to undo/redo. The real bug is actually "some nodes being unselected for no reason".

When snapToGridDuringDrag option is true, after moving selected items only one stays selected. They should stay selected. They do stay selected when snapToGridDuringDrag is false. Can be seen in the below screencast. In the video, for no reason, a node becomes unselected.

chrome_0Ey96FNCea.mp4

Interestingly if you delete line

cytoscape.js-grid-guide/src/snap_during_drag.js

Line 72 in 296adb1

attachedNode.lock();

"being unselected for no reason" bug is not happening. I cannot understand why.

[kubitre]

Nice timing=)