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

Update xkcd.json #423

Closed
wants to merge 2 commits into from
Closed

Update xkcd.json #423

wants to merge 2 commits into from

Conversation

Dankbee
Copy link
Contributor

@Dankbee Dankbee commented Jun 15, 2023

The number count was slightly off. There are actually only 949 colors in this list

The number count was slightly off. There are actually only 949 colors in this list
@hugovk
Copy link
Collaborator

hugovk commented Jun 15, 2023

Hmm, looking at the source: https://blog.xkcd.com/2010/05/03/color-survey-results/

It links to https://xkcd.com/color/rgb/ and https://xkcd.com/color/rgb.txt

The text file has 949, same as this list.

But the grid does have 954.

Here's the missing 5:

  • #011288
  • #087804
  • #16d43f
  • #5f1b6b
  • #c4387f

Would you like to look them up and slot them in?

added 5 additional colors
@Dankbee
Copy link
Contributor Author

Dankbee commented Jun 15, 2023

I put them in on a new pull request :)

@Dankbee Dankbee closed this Jun 15, 2023
@Dankbee Dankbee deleted the patch-1 branch June 15, 2023 18:52
@hugovk hugovk mentioned this pull request Jun 15, 2023
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.

None yet

2 participants