Skip to content

Conversation

@samsamson33
Copy link
Contributor

@samsamson33 samsamson33 commented Feb 27, 2020

This adds the following colors which were missing from the docs (as noted here):

  • BLURPLE
  • GREYPLE
  • DARK_BUT_NOT_BLACK
  • NOT_QUITE_BLACK

Status

  • Code changes have been tested against the Discord API, or there are no code changes
  • I know how to update typings and have done so, or typings don't need updating

Semantic versioning classification:

  • This PR changes the library's interface (methods or parameters added)
    • This PR includes breaking changes (methods removed or renamed, parameters moved or removed)
  • This PR only includes non-code changes, like changes to documentation, README, etc.

This adds the following colors, which were missing from the docs:
- BLURPLE
- GREYPLE
- DARK_BUT_NOT_BLACK
- NOT_QUITE_BLACK
@Moebytes
Copy link
Contributor

Well based on #2729 these are some hidden memes/easter eggs that you can only find by digging into the source code :)

@samsamson33
Copy link
Contributor Author

Oh! Well, they're all official Discord Colors, so if that still stands then I imagine all of them would be Easter eggs. Still, I'll wait for a member to close this if they're intentionally undocumented.

@devsnek
Copy link
Member

devsnek commented Feb 28, 2020

The documented colors are based on the color picker in the role ui in the app. The undocumented ones are based on discord's branding guidelines (which you linked to).

@samsamson33
Copy link
Contributor Author

Right, so should the Discord banding colors be documented, or should they be kept undocumented as a secret of some sort?

@iCrawl iCrawl closed this Mar 1, 2020
@iCrawl iCrawl reopened this Mar 1, 2020
@iCrawl iCrawl changed the title Add missing colors to docs feat(Util): add missing colors to docs Aug 11, 2020
@iCrawl iCrawl merged commit 03580b2 into discordjs:master Aug 11, 2020
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.

5 participants