✨ Add categorical map color palette#6121
Conversation
This stack of pull requests is managed by Graphite. Learn more about stacking. |
|
Quick links (staging server):
Login:
Archive:SVG tester:Number of differences (graphers): 1872 (eb91e9) ❌ Report Edited: 2026-02-24 12:13:44 UTC |
e46a163 to
8aaf475
Compare
43c266b to
69b6516
Compare
8aaf475 to
d96437b
Compare
615ca5c to
f7a6a7c
Compare
69b6516 to
8092da5
Compare
8092da5 to
ed7903d
Compare
f7a6a7c to
e413a2e
Compare
891921a to
9bd4d7e
Compare
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9bd4d7e4ed
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
9bd4d7e to
fbf8a02
Compare
| // We use OwidMapColors instead of the scheme's palette | ||
| // because it includes three additional 'special' colors | ||
| // to be used sparingly when needed (Taupe, Mustard, Tomato) |
There was a problem hiding this comment.
These extra three colors are the main reason the preset colors are customised here, because they're not part of the palette, but should be available to authors in case they need them for specific reasons
| "Sub Saharan Africa": OwidDistinctColors.DarkMauve, | ||
| "South and South East Asia": OwidDistinctColors.OliveGreen, | ||
| "Middle East and North Africa": OwidDistinctColors.Camel, | ||
| "East Asia (Maddison)": OwidDistinctColors.Copper, |
There was a problem hiding this comment.
These ones slipped through somehow: some region names are made up and the 'Maddison' suffix is missing (probably Claude added them and I didn't notice at the time) - fixed now
| "Oceania (UN SDG)": OwidDistinctColors.Turquoise, | ||
| "Sub-Saharan Africa (UN SDG)": OwidDistinctColors.DarkMauve, | ||
|
|
||
| // ILO regions |
| CustomColorSchemes.push(OwidCategoricalMapScheme) | ||
|
|
||
| // Map-specific region colors | ||
| export const MapContinentColors = { |
There was a problem hiding this comment.
These are only used for the region maps in region dods (doesn't make much sense otherwise because these are map-optimised colors for regions that are never plotted on a map...)
fbf8a02 to
d2750b8
Compare
ed7903d to
eb177da
Compare
d2750b8 to
9a4097c
Compare
eb177da to
55c28a4
Compare
Merge activity
|
9a4097c to
9ef45dc
Compare

Resolves #5970
Improves colours for categorical maps
Charts affected by the migration: http://staging-site-categorical-map-color/admin/test/embeds?ids=408,865,1290,4692,4801,4802,4854,5298,5318,5422,5932,5933,6001,6009,6013,6014,6015,6016,6017,6018,6019,6020,6021,6489,7146,7537,7539,7540,7739,7784,7889,7932,8038,8046,8151,8435,8436,8437,8438,8439,8440,8441,8462,8606,8725,8727,8820,8847,8903,8932&comparisonUrl=https%3A%2F%2Fourworldindata.org