search

Swen670Grp1 / cytoscape-platform-tests-js

Reveal.js testing webapp for running platform tests on Cytoscape via CyREST calls
http://chianti.ucsd.edu/~bsettle/cytoscape_platform_test
MIT License
0 stars 0 forks source link

Feature/disable controls #49

Closed OliviaSims21 closed 6 years ago

OliviaSims21 commented 6 years ago

This is for when the error pops up the arrow controls are disabled.

kouissar commented 6 years ago

This looking good. There was one case I detected where controls weren't hidden, see picture ( I believe this is related to the spot for catch all errors (window.onerror) in particular updateError() function. If you address this last piece, this should be good to go.

screen shot 2018-11-28 at 12 49 39 am

OliviaSims21 commented 6 years ago

Made the change Rafik. Please approve.