-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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! |
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: