microsoft / vscode-mssql

Visual Studio Code SQL Server extension.
Other
1.55k stars 458 forks source link

[Rich Experience] Bad Query Result header readability on some themes #18340

Closed atbraz closed 2 weeks ago

atbraz commented 1 month ago

Steps to Reproduce:

  1. Use some non-default themes
  2. Enable in settings.json "mssql.enableRichExperiences": true
  3. Execute any query

On some themes (light and dark) the result header background and foreground colors make for some unreadable combinations.

Examples

Default (Readable)

Dark (Visual Studio)

Image

Light (Visual Studio)

Image

Non-Default Readable

Github Dark

Image

Github Light

Image

Red

Image

Solarized Dark

Image

Non-Default Unreadable (imo)

Monokai Pro (Filter Machine) - or almost any Monokai

Image

Tomorrow Night Blue

Image

Light High Contrast

Image

Ayu Dark

Image

Benjin commented 1 month ago

Thanks for bringing this to our attention!

@cssuh, do you think you'll be able to fix this for November, at least for the Light High Contrast theme? Seems we'll have to delve deeper into the many theme color options to make sure we're using combinations that are visible.

Benjin commented 2 weeks ago

We've fixed many theming issues across the extension with https://github.com/microsoft/vscode-mssql/pull/18347, which will be in our next release. Please open more issues if you find any other places where we have low contrast on some themes!

Monokai Pro (Filter Machine): Image

Tomorrow Night Blue: Image

Light High Constrast: Image

Ayu Dark: Image