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

Possible default for dark mode #6148

Closed
MarcoBarna opened this issue May 22, 2020 · 4 comments
Closed

Possible default for dark mode #6148

MarcoBarna opened this issue May 22, 2020 · 4 comments
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Issue-Question For questions or discussion Product-Terminal The new Windows Terminal. Resolution-Answered Related to questions that have been answered

Comments

@MarcoBarna
Copy link

Thanks Scott Hanselman on YT i have set this theme for all of my shells.
I think it goes really really well with default dark mode.
The setting to get this are:
On every shell settings you have add :

"useAcrylic" : true,
"acrylicOpacity" : 0.85,
"colorScheme" : "Tomorrow Night Eighties"

and here is the color scheme

{
  	"name": "Tomorrow Night Eighties",
  	"black": "#000000",
  	"red": "#f2777a",
  	"green": "#99cc99",
  	"yellow": "#ffcc66",
  	"blue": "#6699cc",
  	"purple": "#cc99cc",
  	"cyan": "#66cccc",
  	"white": "#ffffff",
  	"brightBlack": "#000000",
  	"brightRed": "#f2777a",
  	"brightGreen": "#99cc99",
  	"brightYellow": "#ffcc66",
  	"brightBlue": "#6699cc",
  	"brightPurple": "#cc99cc",
  	"brightCyan": "#66cccc",
  	"brightWhite": "#ffffff",
  	"background": "#2d2d2d",
  	"foreground": "#cccccc"
}

Here is the result
82693215-69dbe180-9c61-11ea-83aa-ca227a404540

@MarcoBarna MarcoBarna added the Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. label May 22, 2020
@ghost ghost added the Needs-Tag-Fix Doesn't match tag requirements label May 22, 2020
@j4james
Copy link
Collaborator

j4james commented May 23, 2020

The problem with this color scheme is that the dark and bright colors are identical. This will inevitably result in situations where text is difficult to see, or even completely invisible. It may be fine for your personal usage, but if we're going to include a color scheme in the app by default, there is an expectation that it's going to work reasonably well for all users. Otherwise it's just going to become another source of bug reports.

@MarcoBarna
Copy link
Author

The problem with this color scheme is that the dark and bright colors are identical. This will inevitably result in situations where text is difficult to see, or even completely invisible. It may be fine for your personal usage, but if we're going to include a color scheme in the app by default, there is an expectation that it's going to work reasonably well for all users. Otherwise it's just going to become another source of bug reports.

Make sense. What about a marketplace like in the settings interface? Something like vscode but for the terminal. In this way it would be fairly easily for everyone and you could also have a sync option?

@j4james
Copy link
Collaborator

j4james commented May 24, 2020

Make sense. What about a marketplace like in the settings interface?

Yeah, I believe something like that is in the long term plans. See issue #4000 (that's mostly about plugin extensions, but I think it should cover themes and color schemes too).

@zadjii-msft
Copy link
Member

Huh, the bot never tagged this with "Needs-Triage", so this got lost. Sorry about that!

@j4james is right - I think the more long-term solution here is we'll have a better story for sharing schemes, themes, and extensions, similar to how VsCode does it. That's still a while off, though we are starting to lay down some foundations in 2.0.

As far as this particular issue is concerned - I think I'm gonna close this mostly based on the fact that this scheme would suffer the same problems that have plagued us with Solarized. If people want it, they're free to add it manually.

Thanks!

@zadjii-msft zadjii-msft added Area-Settings Issues related to settings and customizability, for console or terminal Issue-Question For questions or discussion Product-Terminal The new Windows Terminal. Resolution-Answered Related to questions that have been answered and removed Issue-Feature Complex enough to require an in depth planning process and actual budgeted, scheduled work. labels Jun 23, 2020
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Jun 23, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Issue-Question For questions or discussion Product-Terminal The new Windows Terminal. Resolution-Answered Related to questions that have been answered
Projects
None yet
Development

No branches or pull requests

3 participants