Update Cytoscape JS version to support Z-index

[jlaw9]

When I upload a CYJS and an accompanying style JSON file, the "z-compound-depth" and "z-index-compare" attributes do not seem to be stored or used for drawing the graph. When I download the style JSON file (after uploading), the "z-compound-depth" and "z-index-compare" attributes I had set are gone.

I'm trying to set the parent nodes to draw on top of edges so the text can be seen.

I was able to accomplish it using "z-compound-depth" for the parent node TARGETS in my own javascript file (not in GraphSpace).

[adbharadwaj]

Do you know which version of cytoscape js are you using ?

…

On Jul 31, 2017 8:44 AM, "Jeff Law" ***@***.***> wrote: When I upload a CYJS and an accompanying style JSON file, the "z-compound-depth" and "z-index-compare" attributes do not seem to be stored or used for drawing the graph. When I download the style JSON file (after uploading), the "z-compound-depth" and "z-index-compare" attributes I had set are gone. I'm trying to set the parent nodes to draw on top of edges so the text can be seen. [image: 28386139-9fe22e34-6c98-11e7-801f-74b5440cc016] <https://user-images.githubusercontent.com/7592559/28778037-93e0c00a-75cb-11e7-89d1-b13b83503425.png> I was able to accomplish it using "z-compound-depth" for the parent node TARGETS in my own javascript file (not in GraphSpace). [image: 28386166-b548da16-6c98-11e7-8fd2-33723cf60e4c] <https://user-images.githubusercontent.com/7592559/28778035-92229be4-75cb-11e7-9e31-f498e1e8dd2e.png> — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#295>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADUeBwCWnAPsYzPuIHaRQ53ENj_0w8RUks5sTcwdgaJpZM4OoVBA> .

[adbharadwaj]

This issue is happening because we are using cytoscape.js version 2.7 whereas properties like z-compound-depth are supported by version 3.3 onwards. I tried updating our version to the latest version. But it leads to incompatibility issues with a lot other cytoscape.js extension. Additionally, latest versions do not support event attribute called cyTarget which has led to issues with our popup functionality.

Currently, we cannot update cytoscape.js library.

[adbharadwaj]

This issue requires some investigation into what changes are required to make this update.

[jlaw9]

I made an issue on Cytoscape js to see if the z-index could be set separately for the node and the node label, but unfortunately it won't be implemented (cytoscape/cytoscape.js#1900).