Bad default selection background color with light schemes #11974

Closed
opened 2026-01-31 03:02:55 +00:00 by claunia · 2 comments
Owner

Originally created by @austin-beer on GitHub (Jan 6, 2021).

Environment

Windows build number: 10.0.18363.0
Windows Terminal version: 1.4.3243.0

Steps to reproduce

  1. Update settings.json to use the One Half Light, Solarized Light, or Tango Light color schemes.
  2. Use the mouse to highlight some text.

Expected behavior

It's easy to see which text was selected.

Actual behavior

It's hard to see which text was selected because the default selectionBackground value is white, and that doesn't work well with light color schemes.

Proposed solution

Add selectionBackground values to all of the color schemes in defaults.json (or at least the light color schemes).

Related to

This issue is related to #3326, #3561, and #3580.

Originally created by @austin-beer on GitHub (Jan 6, 2021). # Environment ```none Windows build number: 10.0.18363.0 Windows Terminal version: 1.4.3243.0 ``` # Steps to reproduce 1. Update settings.json to use the `One Half Light`, `Solarized Light`, or `Tango Light` color schemes. 1. Use the mouse to highlight some text. # Expected behavior It's easy to see which text was selected. # Actual behavior It's hard to see which text was selected because the default selectionBackground value is white, and that doesn't work well with light color schemes. # Proposed solution Add selectionBackground values to all of the color schemes in defaults.json (or at least the light color schemes). # Related to This issue is related to #3326, #3561, and #3580.
Author
Owner

@zadjii-msft commented on GitHub (Jan 8, 2021):

Okay, you're not wrong here. Part of the trick with these schemes is that they don't canonically include a selection color. Usually, they're just defined as a set of (foreground, background, 16-color table). The selection color is almost always just set by the terminal emulator. So if we were to add selection BG colors to these schemes, that would be some other customization that we're making to these schemes other than their usual definitions.

Though, I think I agree - until #3561 lands, the experience with these themes is just bad. I wonder if we could just use the "bright black" color for each of these schemes and have it work good enough.

Note

Walkthrough

Go through each of the "light" color schemes in defaults.json, and add a selectionBackground property which is set to the scheme's brightBlack.

This will probably end up in a team discussion, but it'll be helpful to have a concrete example to drive that discussion.

@zadjii-msft commented on GitHub (Jan 8, 2021): Okay, you're not wrong here. Part of the trick with these schemes is that they don't canonically include a selection color. Usually, they're just defined as a set of (foreground, background, 16-color table). The selection color is almost always just set by the terminal emulator. So if we were to add selection BG colors to these schemes, that would be some other customization that we're making to these schemes other than their usual definitions. Though, I think I agree - until #3561 lands, the experience with these themes is just _bad_. I wonder if we could just use the "bright black" color for each of these schemes and have it work _good enough_. > **Note** > ## Walkthrough Go through each of the "light" color schemes in `defaults.json`, and add a `selectionBackground` property which is set to the scheme's `brightBlack`. This will probably end up in a team discussion, but it'll be helpful to have a concrete example to drive that discussion.
Author
Owner

@henryduong26 commented on GitHub (Jun 24, 2021):

  1. Select One Half Light, Solarized Light, or Tango Light color schemes
  2. Input number
    ==> can't see the number inputed

image

@henryduong26 commented on GitHub (Jun 24, 2021): 1. Select One Half Light, Solarized Light, or Tango Light color schemes 2. Input number ==> can't see the number inputed ![image](https://user-images.githubusercontent.com/26895558/123279232-dfd00d80-d531-11eb-9cd0-e78afad5471f.png)
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: starred/terminal#11974