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

Solarized Light's brightWhite and background are set to the same color #2054

Closed
llinfeng opened this issue Jul 21, 2019 · 11 comments
Closed
Labels
Area-Settings Issues related to settings and customizability, for console or terminal Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.
Milestone

Comments

@llinfeng
Copy link

Bug report: Simple man cp does not reveal readable documentation when issued in Windows Terminal. ConEmu does the job properly. Note, I got the same "display problem" across the local machine (Windows 10 + WSL) and a two remote Linux boxes using SSH connection.

For the following screenshot, I used ConEmu on the left and Windows Terminal on the right. Same issue with man ls as shown in the screenshot.

image

@ghost ghost added Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting Needs-Tag-Fix Doesn't match tag requirements labels Jul 21, 2019
@DHowett-MSFT
Copy link
Contributor

Hey, it looks like this is just because Solarized Light's brightWhite and background are set to the same color. That's not right.

@DHowett-MSFT DHowett-MSFT changed the title Bug Report: man cp is not rendered properly, with Ubuntu 18.04 LTS Solarized Light's brightWhite and background are set to the same color Jul 22, 2019
@DHowett-MSFT DHowett-MSFT added Area-Settings Issues related to settings and customizability, for console or terminal Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Product-Terminal The new Windows Terminal. and removed Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Jul 22, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Jul 22, 2019
@DHowett-MSFT
Copy link
Contributor

            "background": "#FDF6E3",
            "brightWhite": "#FDF6E3",
            "name": "Solarized Light",

@zadjii-msft zadjii-msft added this to the Terminal v1.0 milestone Jul 22, 2019
@llinfeng
Copy link
Author

Aha, great catch! I changed the color for "brightWhite" to be the following. Manuals are readable now.

            "brightWhite" : "#1ea2ca",

@dander
Copy link

dander commented Jul 24, 2019

I assume this would apply to these as well?

            "background" : "#002B36",
            "brightBlack" : "#002B36",
            "name" : "Solarized Dark",

@llinfeng
Copy link
Author

llinfeng commented Jul 30, 2019

I am happy with the following color-settings for Solarized Light. It is hand-crafted only to make the terminal more readable. In addition to the brightWhite entry, I also updated the green and blue keys.

        {
            "background" : "#FDF6E3",
            "black" : "#073642",
            "blue" : "#004f87",
            "brightBlack" : "#002B36",
            "brightBlue" : "#839496",
            "brightCyan" : "#93A1A1",
            "brightGreen" : "#586E75",
            "brightPurple" : "#6C71C4",
            "brightRed" : "#CB4B16",
            "brightWhite" : "#1EA2CA",
            "brightYellow" : "#657B83",
            "cyan" : "#2AA198",
            "foreground" : "#073642",
            "green" : "#bac474",
            "name" : "Solarized Light",
            "purple" : "#D33682",
            "red" : "#D30102",
            "white" : "#EEE8D5",
            "yellow" : "#B58900"
        }

I use zsh as my shell, with theme: ZSH_THEME="robbyrussell". With the green entry at its default value, I see something hard to read when listing files in a directory.
image

@ghost ghost added the Needs-Tag-Fix Doesn't match tag requirements label Jul 30, 2019
@DHowett-MSFT
Copy link
Contributor

But now you just don't have green. It seems like a global fix ("delete green") for a local problem ("ls uses blue on green for srwx folders"). That latter bit is configurable with dircolors/LS_COLORS.

@antoineco
Copy link
Contributor

Possibly related to / caused by #293

@kojoru
Copy link
Contributor

kojoru commented Aug 30, 2019

@DHowett-MSFT

Can we reopen this? Right now the experience is like that:

  1. Install git, powershell core and terminal
  2. Set powershell colors to solarized dark
  3. Type "git add --all" in terminal

Expected result: the whole command is visible
Actual result: "--all" is the same colour as the background.

That's definitely caused by having "brightBlack" to be the same as "background", but I'm not sure which colour it should be instead, or whether changing it in the terminal is the right approach.

The problem is definitely there though.

@antoineco
Copy link
Contributor

@kojoru background is supposed to be set to the same color as "bright black" in the Solarized palette (base03). The problem here is the Powershell highlighting, not the Solarized colors.

Please also note that Terminal currently displays a few colors incorrectly due to a bug. See #293 (comment)

I temporarily changed my background color to an almost identical color to get around the issue.

@kojoru
Copy link
Contributor

kojoru commented Sep 2, 2019

@antoineco this is surely a least usability issue though, given both terminal and powershell are microsoft products that are set up using pretty much stock defaults?

As an aside, I'm not actually sure where the color highlighting comes from in powershell. Is that part of https:/PowerShell/PowerShell/? Can I customize that myselft?

@antoineco
Copy link
Contributor

@kojoru I agree that Solarized is always a tricky color scheme to use. Whether in Bash/Zsh or PowerShell, you often have to configure terminal applications to output non-default colors due to Solarized using 16 colors instead of 8 (the standard).

You can change the highlighting of PowerShell commands using Set-PSReadlineOption.

If you also need git colors that work well with Solarized (to add to your .gitconfig file), for example for a prettier git log output, I recommend using these settings.

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 Help Wanted We encourage anyone to jump in on these. Issue-Bug It either shouldn't be doing this or needs an investigation. Needs-Tag-Fix Doesn't match tag requirements Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

6 participants