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

Introduce high contrast light theme #141652

Closed
kupatkar99 opened this issue Jan 27, 2022 · 9 comments · Fixed by #143398
Closed

Introduce high contrast light theme #141652

kupatkar99 opened this issue Jan 27, 2022 · 9 comments · Fixed by #143398
Assignees
Labels
a11ymas Issue from accessibility team accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan themes Color theme issues
Milestone

Comments

@kupatkar99
Copy link

kupatkar99 commented Jan 27, 2022

GitHubTags:

#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #Visual Studio Code Client; #BM-VisualStudioCode-Win32-Nov2021; #Win32; #HighContrast; #WCAG2.1#FTP; #STP; #AINotInScope;

Environment Details:

Application Name: Visual Studio Code Client
Windows Version: Windows 11

Repro Steps:

  1. Turn on high contrast desert theme from settings.
  2. Open Visual studio code editor
  3. Using "Alt+H" key navigate to the "Help" menu bar option .
  4. Using "Down" arrow key navigate to the "Welcome" item.
  5. Make sure you have the following setting in settings.json: "workbench.startupEditor": "gettingStarted"
  6. Using "Alt+H" navigate to "Help" menu bar option and use "down" arrow key to navigate to "Getting started".
  7. Verify whether window is adapting the high contrast desert theme or not.

Actual Result:

Window is not adapting high contrast white theme when high contrast Desert theme is applied.

Expected Result:

Window should adapt the high contrast white theme when high contrast Desert theme is applied.

A11y_VisualStudioCodeClient_Welcome Screen_HighContrast

Introduce high contrast light theme

@JacksonKearl
Copy link
Contributor

This is asking for general support for the variety of different high contrast themes windows supports. Currently we only have the one high contrast theme. cc @misolori and @isidorn any ideas on whether we will support many high contrast themes?

@JacksonKearl JacksonKearl added accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues under-discussion Issue is under discussion for relevance, priority, approach labels Jan 27, 2022
@miguelsolorio
Copy link
Contributor

miguelsolorio commented Jan 27, 2022

+@aeschli, I think we'd to have support in our theme & api structure. We already define a HC color in our color registry so that helps a bit.

But in general, we do not pickup the OS theme (specifically) but we do have the setting window.autoDetectColorScheme to detect dark/light/hc.

@aeschli
Copy link
Contributor

aeschli commented Jan 28, 2022

The Windows High contrast themes consist of 6 different high contrast colors. We have many more colors in our color themes. Deriving a full theme from 6 colors is going to be difficult. If there are some combinations that don't work, if will make the theme useless.
So IMO the user is better served by color theme designed for VS Code.

That said, we should look into a light high contrast theme.

@isidorn isidorn added the a11ymas Issue from accessibility team label Jan 28, 2022
@isidorn
Copy link
Contributor

isidorn commented Jan 28, 2022

Agree with last comment: let's just introduce a light high contrast theme.
I am assigning this to backlog and @misolori can assign to a milestone once there is a passionate designer that would like to add this.

fyi this issue is not from a user but from the testing team

@isidorn isidorn changed the title A11y_VisualStudioCodeClient_Welcome Screen_HighContrast: In high Contrast Desert theme screen in not adapting the feature of desert theme. Introduce high contrast light theme Jan 28, 2022
@isidorn isidorn added feature-request Request for new features or functionality and removed under-discussion Issue is under discussion for relevance, priority, approach labels Jan 28, 2022
@isidorn isidorn added this to the Backlog milestone Jan 28, 2022
@miguelsolorio miguelsolorio added the themes Color theme issues label Jan 28, 2022
@ymoisan ymoisan mentioned this issue Feb 7, 2022
3 tasks
@wesleyboar
Copy link

I have found these other requests/suggestions for at least a High Contrast Light Theme:

@isidorn
Copy link
Contributor

isidorn commented Feb 9, 2022

I see this is on the February plan, thus let me assign it to the February milestone.
@daviddossett let me know how I can best help here. Or reach out to me if you would like to sync.

@isidorn isidorn modified the milestones: Backlog, February 2022 Feb 9, 2022
@daviddossett
Copy link
Contributor

Thanks @isidorn. Working on the first version this week. Will share for feedback shortly 👍

@isidorn
Copy link
Contributor

isidorn commented Feb 21, 2022

@daviddossett I tried this out, and this is a great start!
Can you please create a test plan item for this milestone so we get more eyes on it. Feel free to assign me as one of the testers. Thanks!

@daviddossett
Copy link
Contributor

Thanks for taking a look! Test plan item is here.

@github-actions github-actions bot locked and limited conversation to collaborators Apr 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
a11ymas Issue from accessibility team accessibility Keyboard, mouse, ARIA, vision, screen readers (non-specific) issues feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders on-testplan themes Color theme issues
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants