I looked at the branch. I think it's fine - I just had a few suggestions.
Later on (in a later draft - after these changes are merged) I'd like us to remove references to "cy" from javascript variables, HTML div classes, etc. But I don't think this is important right now – and may never be. More important (probably) would be to remove business logic related to cytoscape version 2 apps, which pre-date Cytoscape's adoption of OSGi and isn't relevant to IGB.
Before submitting a pull request, can you rebase onto the latest master branch? Looks like the last common commit was:
- Narendra Kumar Vankayala cec0368 "Enable social login using Google"
Please check for any possible conflicts. If you rebase and there are some conflicts, you will be in a good position to resolve them – better than me!
Merged changes to 1682.