-
Notifications
You must be signed in to change notification settings - Fork 27.9k
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
fix: restore Dark Modern preformat foreground #197124
Conversation
The real fix is to update index.html to use the new preformat-background colors, but that's a larger change than we should take for endgame, so this commit just reverts to the previous colors we had for the release
b7831cd
to
1ccbfc5
Compare
@@ -113,7 +113,7 @@ | |||
"textCodeBlock.background": "#2B2B2B", | |||
"textLink.activeForeground": "#4daafc", | |||
"textLink.foreground": "#4daafc", | |||
"textPreformat.foreground": "#D0D0D0", | |||
"textPreformat.foreground": "#D7BA7D", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required for code
blocks in Markdown previews in Dark Modern to render in a distinct color
@@ -131,7 +131,7 @@ | |||
"textCodeBlock.background": "#F8F8F8", | |||
"textLink.activeForeground": "#005FB8", | |||
"textLink.foreground": "#005FB8", | |||
"textPreformat.foreground": "#3B3B3B", | |||
"textPreformat.foreground": "#A31515", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required for code
blocks in Markdown previews in Light Modern to render in a distinct color
@@ -227,7 +227,7 @@ export const selectionBackground = registerColor('selection.background', { light | |||
export const textSeparatorForeground = registerColor('textSeparator.foreground', { light: '#0000002e', dark: '#ffffff2e', hcDark: Color.black, hcLight: '#292929' }, nls.localize('textSeparatorForeground', "Color for text separators.")); | |||
export const textLinkForeground = registerColor('textLink.foreground', { light: '#006AB1', dark: '#3794FF', hcDark: '#3794FF', hcLight: '#0F4A85' }, nls.localize('textLinkForeground', "Foreground color for links in text.")); | |||
export const textLinkActiveForeground = registerColor('textLink.activeForeground', { light: '#006AB1', dark: '#3794FF', hcDark: '#3794FF', hcLight: '#0F4A85' }, nls.localize('textLinkActiveForeground', "Foreground color for links in text when clicked on and on mouse hover.")); | |||
export const textPreformatForeground = registerColor('textPreformat.foreground', { light: '#A31515', dark: '#D7BA7D', hcDark: '#000000', hcLight: '#FFFFFF' }, nls.localize('textPreformatForeground', "Foreground color for preformatted text segments.")); | |||
export const textPreformatForeground = registerColor('textPreformat.foreground', { light: '#A31515', dark: '#D7BA7D', hcDark: '#D7BA7D', hcLight: '#292929' }, nls.localize('textPreformatForeground', "Foreground color for preformatted text segments.")); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required for code
blocks in Markdown previews in HC themes to be visible at all
@@ -244,7 +244,6 @@ | |||
font-family: var(--monaco-monospace-font); | |||
font-size: 12px; | |||
color: var(--vscode-textPreformat-foreground); | |||
background-color: var(--vscode-textPreformat-background); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required to ensure that code
blocks are visible in chat after updating vscode-textPreformat-foreground
@@ -533,7 +533,6 @@ | |||
font-family: var(--monaco-monospace-font); | |||
font-size: 11px; | |||
color: var(--vscode-textPreformat-foreground); | |||
background-color: var(--vscode-textPreformat-background); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is required to ensure that code
blocks are visible in the settings UI after updating vscode-textPreformat-foreground
No description provided.