search

iVis-at-Bilkent / cytoscape.js-clipboard

A Cytoscape.js extension to provide copy-paste utilities
MIT License
8 stars 4 forks source link

Do not need jQuery #2

Closed lukejpreston closed 7 years ago

lukejpreston commented 7 years ago

options wasn't being used and you can use concat instead of jQueries extend. This means you don't need the full jQuery library only for the extend function. Also remove options from the README.md because it did nothing.

kinimesi commented 7 years ago

@lukejpreston see https://github.com/iVis-at-Bilkent/cytoscape.js-clipboard/issues/1#issuecomment-306781353

Your modification breaks up undo/redo functionality: copy a node, then paste it twice. Moreover after pasting twice, the pasted nodes disappear/reappear randomly when clicked on.