Skip to content

Use default terminal colors in debug console and add seperate high-intensity ANSI colors#70969

Closed
iansan5653 wants to merge 2 commits intomicrosoft:masterfrom
iansan5653:terminal-colors-in-debug
Closed

Use default terminal colors in debug console and add seperate high-intensity ANSI colors#70969
iansan5653 wants to merge 2 commits intomicrosoft:masterfrom
iansan5653:terminal-colors-in-debug

Conversation

@iansan5653
Copy link
Contributor

This updates the default debug console colors to match the default terminal colors found here: https://github.com/Microsoft/vscode/blob/1f64c126721a66f30a7dcc3ac9e605e8115bcca7/src/vs/workbench/contrib/terminal/common/terminalColorRegistry.ts#L40

It also adds separate bright/high-intensity colors to close #63986.

The existing colors have been in place since VSCode was first put on GitHub, but they seem to be pretty arbitrarily chosen. They're muted and hard to read, and they are inconsistent with the rest of the editor.

This is a precursor to a future pull request in which I'll add functionality to update the colors based on the settings made by the current theme per #42388 (I'd still leave this CSS here as defaults though).

@iansan5653 iansan5653 marked this pull request as ready for review March 22, 2019 17:24
@isidorn isidorn self-assigned this Mar 25, 2019
@isidorn isidorn added this to the March 2019 milestone Mar 25, 2019
@isidorn isidorn added the debug Debug viewlet, configurations, breakpoints, adapter issues label Mar 25, 2019
@isidorn isidorn modified the milestones: March 2019, April 2019 Mar 25, 2019
@isidorn
Copy link
Collaborator

isidorn commented Mar 25, 2019

Same as other PR, this is great work. But we will merge in next week

fyi @misolori for color changes in the debug console. I think these will be improvments but I am not sure if we need to re-check colors in debug console for contrast ratios for accessibility once we merge this in.

@iansan5653
Copy link
Contributor Author

@isidorn I'm actually thinking it would be better to remove the CSS rules altogether and just pull the colors from terminalColorRegistry, applying them as inline styles. Doing so would eliminate the need for the CSS since they would already fall back to the defaults defined in that file. If that sounds good to you, we can close this PR and wait until I have the new behavior ready. If the new PR is rejected for whatever reason, we can reopen this one.

@isidorn
Copy link
Collaborator

isidorn commented Mar 26, 2019

@iansan5653 not having that duplicated makes good sense to me. Let's close this PR and wait for the other one.

@isidorn isidorn closed this Mar 26, 2019
@github-actions github-actions bot locked and limited conversation to collaborators Mar 27, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

debug Debug viewlet, configurations, breakpoints, adapter issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support high intensity ansi colours in debug console

2 participants