Skip to content
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

Selection Color should invert selected text color #3561

Open
Tracked by #13392
javierdlg opened this issue Nov 13, 2019 · 5 comments
Open
Tracked by #13392

Selection Color should invert selected text color #3561

javierdlg opened this issue Nov 13, 2019 · 5 comments
Labels
Area-Fonts Related to the font Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Help Wanted We encourage anyone to jump in on these. Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal.
Milestone

Comments

@javierdlg
Copy link
Member

javierdlg commented Nov 13, 2019

Environment

Windows build number: 10.0.18363.0
Windows Terminal version (if applicable): All

Steps to reproduce

  1. Open the terminal.
  2. Select some text.

Expected behavior

Selected text should invert the text colors:

Actual behavior

Text maintains it's original color which leads to contrast issues.
image

Notes

Powershell and CMD both invert the text colors when selected, which would fix the contrast ratio accessibility issue present in the terminal:
image

@javierdlg javierdlg added Area-Fonts Related to the font Issue-Bug It either shouldn't be doing this or needs an investigation. labels Nov 13, 2019
@ghost ghost added Needs-Tag-Fix Doesn't match tag requirements Needs-Triage It's a new issue that the core contributor team needs to triage at the next triage meeting labels Nov 13, 2019
@zadjii-msft
Copy link
Member

oh no

This is hard to very hard on the scale of tasks, unfortunately.

GDI has a very handy function for inverting the colors of a region of the window. unfortunately, DX has so such equivalent function. I believe @miniksa has said something in that past about rendering the frame, then specially inverting the selected area in another rendering pass? He can elaborate more.

@zadjii-msft zadjii-msft added Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal. and removed Issue-Bug It either shouldn't be doing this or needs an investigation. labels Nov 13, 2019
@ghost ghost removed the Needs-Tag-Fix Doesn't match tag requirements label Nov 13, 2019
@zadjii-msft zadjii-msft added this to the Terminal Backlog milestone Nov 13, 2019
@javierdlg
Copy link
Member Author

How about being able to choose the selected text foreground color? This could potentially solve the contrast ratio issue on selected text and not force us to vet all the colors against two background colors for every theme.

@miniksa
Copy link
Member

miniksa commented Nov 13, 2019

Didn't #3471 committed today literally just do this?

@mdtauk
Copy link

mdtauk commented Nov 13, 2019

Didn't #3471 committed today literally just do this?

It only includes a Selection Background colour. I did make the suggestion there should also be a Selection Foreground colour.

@DHowett-MSFT
Copy link
Contributor

We're open to contributions to our DX renderer. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area-Fonts Related to the font Area-Rendering Text rendering, emoji, complex glyph & font-fallback issues Help Wanted We encourage anyone to jump in on these. Issue-Task It's a feature request, but it doesn't really need a major design. Product-Terminal The new Windows Terminal.
Projects
None yet
Development

No branches or pull requests

5 participants