Skip to content

fix: remove transparent quickInput.background in Dark Modern theme (fix #293097)#293137

Open
murataslan1 wants to merge 1 commit intomicrosoft:mainfrom
murataslan1:fix/issue-293097-dark-modern-transparency
Open

fix: remove transparent quickInput.background in Dark Modern theme (fix #293097)#293137
murataslan1 wants to merge 1 commit intomicrosoft:mainfrom
murataslan1:fix/issue-293097-dark-modern-transparency

Conversation

@murataslan1
Copy link
Copy Markdown
Contributor

PR Title

fix: remove transparent quickInput.background in Dark Modern theme (fix #293097)

Description

Fixes issue #293097 where the Command Palette (Quick Pick) became transparent in the "Dark Modern" theme, making it unreadable.

The issue was caused by an explicit definition of "quickInput.background": "#222222" in extensions/theme-defaults/themes/dark_modern.json. Although the color value itself is opaque, its presence seemed to interfere with the expected widget styling or fallback behavior.

By removing this explicit definition, the theme now correctly inherits editorWidget.background from the base theme (or default behavior), which resolves to #202020 in this context. This matches the behavior of "Dark+" and restores the expected opacity.

Changes:

  • Removed quickInput.background from extensions/theme-defaults/themes/dark_modern.json.

Related Issue

Fixes #293097

Copilot AI review requested due to automatic review settings February 5, 2026 15:33
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR addresses issue #293097 where the Command Palette (Quick Pick) became transparent in the "Dark Modern" theme, making it unreadable when viewing bright content. The fix removes the explicit quickInput.background color definition, allowing the theme to fall back to the inherited editorWidget.background value.

Changes:

  • Removed quickInput.background from the Dark Modern theme to fix transparency issues

@@ -73,7 +73,6 @@
"peekViewResult.matchHighlightBackground": "#BB800966",
"pickerGroup.border": "#3C3C3C",
"progressBar.background": "#0078D4",
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The removal of quickInput.background from this theme file should be accompanied by a corresponding update to COLOR_THEME_DARK_INITIAL_COLORS in src/vs/workbench/services/themes/common/workbenchThemeService.ts. Currently, the initial colors (used to prevent paint flashing during startup) still define quickInput.background as #222222, which creates an inconsistency where the initial rendering will use #222222 but after the theme loads it will fall back to editorWidget.background (#202020). This inconsistency could cause a visual flash when the theme fully loads.

Suggested change
"progressBar.background": "#0078D4",
"progressBar.background": "#0078D4",
"quickInput.background": "#222222",

Copilot uses AI. Check for mistakes.
"pickerGroup.border": "#3C3C3C",
"progressBar.background": "#0078D4",
"quickInput.background": "#222222",
"quickInput.foreground": "#CCCCCC",
Copy link

Copilot AI Feb 5, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

For consistency, consider also removing quickInput.background from light_modern.json since it's set to #F8F8F8 which is identical to editorWidget.background in that theme. This would make both modern themes follow the same pattern of inheriting from editorWidget.background. Note that the corresponding COLOR_THEME_LIGHT_INITIAL_COLORS would also need to be updated.

Suggested change
"quickInput.foreground": "#CCCCCC",

Copilot uses AI. Check for mistakes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Command palette in new dark theme unreadable if you have something bright open

3 participants