-
Notifications
You must be signed in to change notification settings - Fork 198
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
Damovisa/dark high contrast inline code block #395
Conversation
🦋 Changeset detectedLatest commit: b7991c4 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Colors changed--- base/themes/dark-colorblind.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/dark-colorblind.json 2024-10-03 02:06:42.670144381 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#8b949e",
+ "textPreformat.background": "#6e768166",
"textSeparator.foreground": "#21262d",
--- base/themes/dark-default.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/dark-default.json 2024-10-03 02:06:42.670144381 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#7d8590",
+ "textPreformat.background": "#6e768166",
"textSeparator.foreground": "#21262d",
--- base/themes/dark-dimmed.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/dark-dimmed.json 2024-10-03 02:06:42.670144381 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#768390",
+ "textPreformat.background": "#636e7b66",
"textSeparator.foreground": "#373e47",
--- base/themes/dark-high-contrast.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/dark-high-contrast.json 2024-10-03 02:06:42.670144381 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#f0f3f6",
+ "textPreformat.background": "#9ea7b366",
"textSeparator.foreground": "#7a828e",
--- base/themes/light-colorblind.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/light-colorblind.json 2024-10-03 02:06:42.669144375 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#57606a",
+ "textPreformat.background": "#afb8c133",
"textSeparator.foreground": "#d8dee4",
--- base/themes/light-default.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/light-default.json 2024-10-03 02:06:42.669144375 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#656d76",
+ "textPreformat.background": "#afb8c133",
"textSeparator.foreground": "#d8dee4",
--- base/themes/light-high-contrast.json 2024-10-03 02:06:44.674164603 +0000
+++ themes/light-high-contrast.json 2024-10-03 02:06:42.669144375 +0000
@@ -13,2 +13,3 @@
"textPreformat.foreground": "#0e1116",
+ "textPreformat.background": "#acb6c033",
"textSeparator.foreground": "#88929d", |
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.
Thanks ❤️
Using the same as for textCodeBlock.background
looks good to me. 👍
Describe your changes here.
Inline highlighted text in the Copilot Chat response was unreadable in the GitHub Dark High Contrast theme. This PR adds a missing background color for the
textPreformat.background
variable.Closes #380
Screenshots
Before:
After:
Merge checklist
GitHub Light Default
themeTake a look at the Contribute section for more information on how test your changes locally.