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

✨ reordered owid distinct colors #4003

Merged
merged 1 commit into from
Oct 10, 2024
Merged

Conversation

danyx23
Copy link
Contributor

@danyx23 danyx23 commented Oct 3, 2024

This fixes #3898

@owidbot
Copy link
Contributor

owidbot commented Oct 3, 2024

Quick links (staging server):

Site Admin Wizard

Login: ssh owid@staging-site-owid-distinct-colors-reordered

SVG tester:

Number of differences (default views): 736 (7a96a0) ❌
Number of differences (all views): 370 (8db9d0) ❌

Edited: 2024-10-03 18:02:50 UTC
Execution time: 1.20 seconds

@lucasrodes
Copy link
Member

lucasrodes commented Oct 4, 2024

hey @danyx23,
Thanks for the rapid implementation! I think this fixes a substantial part of #3898, but the initial confusion about how colors are chosen is still not resolved.

I think I understood how colors are picked when the user selects/unselects entities in charts. I wonder if this is the way to go. Elaborated on this here: #3898 (comment)

Copy link
Contributor Author

danyx23 commented Oct 4, 2024

@lucasrodes yes, this only implements a better order of colors which makes the default color choices on many of our charts substantially better.

I see your other point about how selecting and deselecting is a bit weird but there are several other concerns that play into that part of the codebase (e.g. persistence of colors on entities coming in via facetting or explorers). It's possible that we can improve that in the future but this is not a quick fix.

@danyx23 danyx23 merged commit 61de17b into master Oct 10, 2024
22 of 24 checks passed
@danyx23 danyx23 deleted the owid-distinct-colors-reordered branch October 10, 2024 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bug: colors in line charts are often very similar
3 participants