kaiusl / pastel_evening_vscode

Dark VS Code theme with pastel colors and a slight blue hint to the editor.
https://marketplace.visualstudio.com/items?itemName=kaiusl.pastel-evening-theme
MIT License
2 stars 0 forks source link

Please improve contrast on text selection #1

Closed Meligy closed 8 months ago

Meligy commented 8 months ago

Make sure to use the latest version of the theme before adding a report. ✅ v0.3.0 Describe the bug I would like the text selection background to be a bit easier to distinguish.

Here I searched for "workbench", selected "Pastel":

image

Now with the same selection, I just hovered over the closing double quote of the string value:

image

Another example, this time selecting "centeredLayoutAutoResize":

image

More examples:

Selecting "get":

image

(It's worth noting that the contrast looks more obvious in the screenshot than on real screen for some reason)

Selecting "the":

image

Extra information:

If you can think of any other relevant information to help find and fix the bug quickly, please add it.

Again: It's worth noting that the contrast looks more obvious in the screenshot than on real screen for some reason. So, I struggle more using the screen than it seems in them.

Meligy commented 8 months ago

Using the template, I forgot to first say thanks a lot for the great theme.

I have found very few extensions other than the one that ships with the C# extension (not even the VS Code default ones) to have a proper highlighting of interfaces vs classes etc.

kaiusl commented 8 months ago

I see what you mean but I'm not quite sold on changing it at the moment. The selection background was meant to be relatively subtle.

However you can always change the colors to your liking through "settings.json". To change the selection background add the following lines with any color (ones in the snippet are the default ones):

"workbench.colorCustomizations": {
    "[Pastel Evening Dark]": {
        "editor.selectionBackground": "#324454ff",
        "editor.inactiveSelectionBackground": "#2b3947ff",
    }
},
Meligy commented 8 months ago

Thanks for the guidance.

As I report this, I realise there are 2 different issues here:

  1. How the selection is subtle, which can be argued whether it's a bug or a matter of taste. That was my intended purpose of the issue.

  2. How the selection color matches the hover effect of many other tokens. Like a reference match or a string highlight.
    See the 2nd picture in my issue. Here's another recreation: image You can't tell that I only selected the first word of the string ("Pastel").
    Another example: image In this example, I selected Extensions in settingsSync.ignoredExtensions. I also hovered over hediet.vscode-drawio.theme without a selection.

Meligy commented 8 months ago

Side note:

The "FF" at the end of editor.inactiveSelectionBackground triggered a warning from VS Code:

image

Changing it to even "FE" makes the warning goes away, but I was not able to test it anyway.

Not saying it's something bad or anything. Just thought you'd want to know it's a thing!

Thanks again.

kaiusl commented 8 months ago

2. How the selection color matches the hover effect of many other tokens. Like a reference match or a string highlight.

Ah, that's indeed an issue. I'll try to figure something out for them. Meanwhile if you want to change it yourself the relevant keys to modify are

"workbench.colorCustomizations": {
    "[Pastel Evening Dark]": {
        "editor.wordHighlightStrongBackground": "#324454ff",
        "editor.wordHighlightBackground": "#324454ff",
        "editor.hoverHighlightBackground": "#324454ff",
        "editor.symbolHighlightBackground": "#324454ff",
        "editor.selectionHighlightBackground": "#324454ff",
        "editor.rangeHighlightBackground": "#324454ff",
        "editor.findRangeHighlightBackground": "#324454ff",
        "editor.focusedStackFrameHighlightBackground": "#324454ff",
        "editor.snippetTabstopHighlightBackground": "#324454ff",
        "editor.stackFrameHighlightBackground": "#324454ff",
        "editor.snippetFinalTabstopHighlightBackground": "#324454ff",
    }
}
kaiusl commented 8 months ago

Side note:

The "FF" at the end of editor.inactiveSelectionBackground triggered a warning from VS Code:

image

Changing it to even "FE" makes the warning goes away, but I was not able to test it anyway.

Not saying it's something bad or anything. Just thought you'd want to know it's a thing!

Thanks, hadn't noticed that before. The property description says: "Color of the selection in an inactive editor. The color must not be opaque so as not to hide underlying decorations." but on a quick test I couldn't see the decorations disappearing. Nonetheless probably should change the color to something non opaque in future versions.

kaiusl commented 8 months ago

New v0.3.1 released which fixes the opaque selection colors and conflicting selection and editor highlight colors. The selection color itself remains roughly as it was.

Meligy commented 8 months ago

Thanks a lot.