search

cytoscape / cytoscape.js

Graph theory (network) library for visualisation and analysis
https://js.cytoscape.org
MIT License
10.12k stars 1.64k forks source link

List extension : cyto_cxtmenu #2061

Closed callin2 closed 6 years ago

callin2 commented 6 years ago

https://github.com/callin2/cyto_cxtmenu Context menu for cytoscape.js. nested and dynamic submenu support.

maxkfranz commented 6 years ago

Would you please structure your extension a bit more like the others?

  1. There is no demo.
  2. There is no documentation in the readme.
  3. The source code annotations are not in English. Most programmers generally know at least basic English, so English should be used for all documentation, comments, etc.
  4. It would be nice to have a screenshot in the readme, since this is a UI extension.
  5. It's not published to npm.

Several of these could be easily addressed by using the scaffolder: https://github.com/cytoscape/slush-cytoscape-extension

maxkfranz commented 6 years ago

This can be re-opened once the above points are addressed.