cytoscape / cytoscape-explore

Network visualization webapp.
MIT License
12 stars 4 forks source link

Basic support for style bypasses in the style picker. #64

Closed mikekucera closed 3 years ago

mikekucera commented 3 years ago

Allows style bypasses to be set when one or more nodes/edges are selected. Refs #52.

This does not implement a global "mode" for bypasses. It just switches the style picker to set or clear bypasses on selected elements.

I don't think this is final, any feedback on UI improvements is appreciated. Also, should we use the term "bypass" in the UI, or something else?

maxkfranz commented 3 years ago

I don't think this is final, any feedback on UI improvements is appreciated.

Let's use the issues to discuss UI. I have some ideas on how to simplify things.

Also, should we use the term "bypass" in the UI, or something else?

It's fine for now.

I think as long as it's coupled to selection, users will just get it without using the word "bypass". Drawing tools work based on selection. The main thing for this PR is to make sure that the API is working correctly for the kinds of actions we want to perform.

I'll merge this in.

Let's discuss the UI in #52