Skip to content
This repository has been archived by the owner on Oct 7, 2021. It is now read-only.

Add edge status colors #29

Merged
merged 1 commit into from
Aug 15, 2018
Merged

Conversation

cshinn
Copy link
Contributor

@cshinn cshinn commented Aug 13, 2018

Adds recommendations for graph edge status colors
Updates pf-black-400 to pf-black-500 to increase contrast

closes #24


### Unused
![unused edges](img/edge-status-unused.png)
- Grey Stroke: #8b8d8f (pf-black-500)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What is the use case for "unused"?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In the current legend, that's labeled with "no traffic yet"

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Isn't Idle == no traffic?
If unused means no traffic, what idle means then?

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Edges to an unused node have no traffic (i.e. the dotted lines). They exist only when there are multiple versions and we group it with the other version nodes. Having the same color for
"idle" edges (i.e. an edge that has shown traffic for other time ranges, but not for the current one) and unused edges (has never seen traffic), makes sense.

As for the legend, I think this will be a high priority for Sprint 10, and it should be replaced with a dialog with much more detailed descriptions.

@abonas
Copy link
Contributor

abonas commented Aug 13, 2018

is this the resolution for https://github.com/kiali/kiali-design/issues/24? if so, they should be linked (and then #24 will auto close when this one is merged)

@abonas
Copy link
Contributor

abonas commented Aug 13, 2018

@mtho11 fyi

@mtho11
Copy link

mtho11 commented Aug 13, 2018

@jshaughn if you are taking https://issues.jboss.org/browse/KIALI-1194 you will need to add this

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Colors design is needed for links
5 participants