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 Dark theme renders certain commands/text unreadable. #5896

Closed
MiniFlames opened this issue May 13, 2020 · 6 comments
Closed
Labels
Issue-Question For questions or discussion Needs-Tag-Fix Doesn't match tag requirements Resolution-Answered Related to questions that have been answered

Comments

@MiniFlames
Copy link

MiniFlames commented May 13, 2020

Certain text is unreadable when using the 'Solarized Dark' theme, for example the following command in Powershell

pktmon start --etw -m real-time

is rendered like this:

image

Thus far I've only noticed the issue with Powershell commands.


Windows
Terminal Version(s): 0.11.1121.0 and 0.11.1333.0
Powershell Version: Powershell Core 7.0.0
Windows Version: Windows 10 - Insider Preview (Fast Ring), build 19268.

@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 May 13, 2020
@MiniFlames MiniFlames changed the title Solarized Dark theme renders certain commands/text unreadable. 🐛 Solarized Dark theme renders certain commands/text unreadable. May 13, 2020
@skyline75489
Copy link
Collaborator

I've experienced this once with PSCore. Changing the color setting of PSCore itself (using Set-PSReadlineOption) effectively fix this. I'm not sure what we can do (or should do) on the terminal side.

@jdhitsolutions
Copy link

I have found that when using this theme I have to make sure not to set useAcrylic to false and remove the PSReadline module. The problem is the conflict between PSReadline options and the WT theme. I'm not sure that can be fixed. I'd say your best solution is to use Set-PSReadlineOption to adjust colors when using the Solarized Dark theme.

@jdhitsolutions
Copy link

This is the PowerShell script I run when I need to adjust colors.

#Update-PSReadline.ps1
# https://en.wikipedia.org/wiki/ANSI_escape_code

if ($IsCoreCLR) {
    $esc = "`e"
}
else {
    $esc = $([char]0x1b)
}

Set-PSReadLineOption -Colors @{
    Parameter = "$esc[96m"
    Operator  = "$esc[38;5;47m"
    comment   = "$esc[92m"
    String    = "$esc[38;5;51m"
}

@j4james
Copy link
Collaborator

j4james commented May 14, 2020

This is a very old bug in the Solarized color scheme (altercation/solarized#220).

@DHowett-MSFT
Copy link
Contributor

Yeah, this one is unfortunately by design. Solarized was not a good color scheme for us to ship by default, but here we are on the doorstep of v1 being sad about it. ☹️

@DHowett-MSFT
Copy link
Contributor

FWIW: some combination of #2661 and #293 are at play here.

@DHowett-MSFT DHowett-MSFT added Issue-Question For questions or discussion Resolution-Answered Related to questions that have been answered and removed Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels May 15, 2020
@ghost ghost added the Needs-Tag-Fix Doesn't match tag requirements label May 15, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Issue-Question For questions or discussion Needs-Tag-Fix Doesn't match tag requirements Resolution-Answered Related to questions that have been answered
Projects
None yet
Development

No branches or pull requests

5 participants