Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make colors different when graphing 2 different lines #2662

Closed
macobo opened this issue Dec 4, 2020 · 10 comments
Closed

Make colors different when graphing 2 different lines #2662

macobo opened this issue Dec 4, 2020 · 10 comments
Labels
enhancement New feature or request feature/trends Feature Tag: Trends good first issue Good for newcomers

Comments

@macobo
Copy link
Contributor

macobo commented Dec 4, 2020

Is your feature request related to a problem?

When creating a graph with two actions/events or breakdown generates 2 distinct groups, the colors used always seem to be blue.

Example:
Screenshot from 2020-12-04 10-56-22

Describe the solution you'd like

Use distinguishable colors.

Describe alternatives you've considered

Hovering over things

Additional context

Came up during #2560

Thank you for your feature request – we love each and every one!

@macobo macobo added enhancement New feature or request UI/UX feature/trends Feature Tag: Trends labels Dec 4, 2020
@timgl timgl added the good first issue Good for newcomers label Dec 4, 2020
@paolodamico
Copy link
Contributor

+1 from customer call this week

@paolodamico
Copy link
Contributor

@lottiecoxon might be worth giving some deep thought as to which colors we should use here. From a customer call several weeks ago we also got the feedback that the colors we're using don't have a good enough contrast ratio and are particularly hard to read for color-blind people, and particularly relevant now that you're working on a new palette.

@paolodamico
Copy link
Contributor

Related to #1763.

We need to address this soon, check out this example.

@lottiecoxon
Copy link
Contributor

Hey @paolodamico sorry for the really long delay, I just realised there were a lot of mentions on GitHub that I have missed, so I will get to these all today!

I found this article online so will work on a colour blind friendly scheme - https://venngage.com/blog/color-blind-friendly-palette/

@lottiecoxon
Copy link
Contributor

Here are 4 colour schemes, laid out so you can see the 'normal' colour vision on the top level, and below how these colours are seen by people with the two most common colour blindness types.

Colour Blind

@mariusandra
Copy link
Collaborator

Hey, this used to be fine, but something messed it up.

For some context, we can't just hardcode colors as we'd go crazy with all the different options. We have five different backgrounds (black, white, blue, green, purple) and we need colors on all of them that look good with the background, yet different from each other. Plus we need about 20 unique colors for each of them.

I had created a system to generate any number of relatively unique colors for every background we can throw at it, but this doesn't seem to work anymore. Now every line is just blue. Thus it's a bug that needs to be fixed. I might have a look a bit later today.

@mariusandra
Copy link
Collaborator

All backgrounds were presumed to be white due to a bug. Now the lines on the colored backgrounds look better, though this probably doesn't fix the initial issue.
#2911

@mariusandra
Copy link
Collaborator

I found a fix for the two blue lines problem as well: 4c781df

Turned out to be a 🤦

@paolodamico
Copy link
Contributor

I think the immediate impact has been addressed. However I do think we still need to do a better job setting high-contrast colors, which are also as different from each other as possible. See below a graph with a simple breakdown, the gray ones are very hard to see (and it's definitely not-accessible). I do think this is a matter more on the design side than engineering.

@Twixes
Copy link
Member

Twixes commented Aug 3, 2021

New graphs have this.

@Twixes Twixes closed this as completed Aug 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request feature/trends Feature Tag: Trends good first issue Good for newcomers
Projects
None yet
Development

No branches or pull requests

6 participants