-
Notifications
You must be signed in to change notification settings - Fork 8.5k
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
Author a new default color scheme (to replace Campbell) #17818
Comments
Current theme: {
"name": "Prototype Bright",
"background": "#000000",
"foreground": "#bebebe",
"cursorColor": "#ffffff",
"selectionBackground": "#92a4fd",
"black": "#000000",
"red": "#be2c21",
"green": "#3fae3a",
"yellow": "#be9a4a",
"blue": "#204dbe",
"purple": "#bb54be",
"cyan": "#00a7b2",
"white": "#bebebe",
"brightBlack": "#808080",
"brightRed": "#ff3e30",
"brightGreen": "#58ea51",
"brightYellow": "#ffc944",
"brightBlue": "#2f6aff",
"brightPurple": "#fc74ff",
"brightCyan": "#00e1f0",
"brightWhite": "#ffffff"
} |
Here's my testing code I wrote: https://refined-github-html-preview.kidonng.workers.dev/microsoft/terminal/raw/dev/lhecker/theme-quality/doc/theme_colors.html My primary focus is on the first scheme there: "Prototype Bright". Here's how I created the colors:
|
As I wrote elsewhere a long time ago... When changing the default colors, please consider the following:
"Prototype Bright" looks neat, but let's be honest, it is not really a 16 color palette, it is a 9 color palette. |
For reference, here's my current proposal and the NT ("Vintage") theme side by side: Crucially, I personally believe, your 5th point runs somewhat counter with the arguments in #1260: The I tried experimenting with hue shifts just now to make the colors more distinct and it looks somewhat ugly unfortunately in my opinion. If you have any advice how I can design the palette better, please let me know! |
Campbell has been the default color scheme for a long time now, but it has quite some issues with hue and chroma. This PR introduces a new scheme which was created using the Oklab color space to find colors with maximal distance to each other and well distributed and consistent hue and chroma. Because of this, I've named the scheme after the creator of Oklab. Closes #17818 (cherry picked from commit e60acbc) Service-Card-Id: PVTI_lADOAF3p4s4AxadtzgW-IC0 Service-Version: 1.23
Campbell has been the default color scheme for a long time now, but it has quite some issues with hue and chroma. This PR introduces a new scheme which was created using the Oklab color space to find colors with maximal distance to each other and well distributed and consistent hue and chroma. Because of this, I've named the scheme after the creator of Oklab. Closes #17818 (cherry picked from commit e60acbc) Service-Card-Id: PVTI_lADOAF3p4s4AmhmQzgW-IDA Service-Version: 1.22
Campbell has been the default color scheme for a long time now, but it has quite some issues with hue and chroma. This PR introduces a new scheme which was created using the Oklab color space to find colors with maximal distance to each other and well distributed and consistent hue and chroma. Because of this, I've named the scheme after the creator of Oklab. Closes #17818 (cherry picked from commit e60acbc) Service-Card-Id: PVTI_lADOAF3p4s4AmhmszgW-IC8 Service-Version: 1.21
There's been some internal discussions around replacing Campbell as the default scheme.
I'm filing this to track that discussion
The text was updated successfully, but these errors were encountered: