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

ContentDialog Text Color Contrast #2354

Closed
carlos-zamora opened this issue Apr 29, 2020 · 8 comments
Closed

ContentDialog Text Color Contrast #2354

carlos-zamora opened this issue Apr 29, 2020 · 8 comments
Assignees
Labels
area-Dialogs team-Controls Issue for the Controls team

Comments

@carlos-zamora
Copy link
Member

Describe the bug
The color contrast of hyperlinks and text in a ContentDialog does not meet the color contrast ratio of 4.5:1 for accessibility.

microsoft/terminal#5395 is dependent on a fix for this issue.

Steps to reproduce the bug
This can be seen in Windows Terminal's TerminalPage.xaml , but this just uses the built-in theme dictionaries.

Steps to reproduce the behavior:

  1. Open your app of choice that uses a ContentDialog using HyperlinkButtons
  2. Open Accessibility Insights (download link here) to check the color contrast ratio
  3. Verify whether the color contrast ratio for the SplitButton control meets the 4.5:1 contrast ratio

Expected behavior
In Windows Terminal, the links present in the "About" pop up window and the text to the back ground color in the "OK" button should meet the color contrast ratio 4.5:1

Screenshots
image

image

Version Info

NuGet package version:

Windows 10 version Saw the problem?
Insider Build (xxxxx)
November 2019 Update (18363)
May 2019 Update (18362)
October 2018 Update (17763)
April 2018 Update (17134)
Fall Creators Update (16299)
Creators Update (15063)
Device form factor Saw the problem?
Desktop
Mobile
Xbox
Surface Hub
IoT
@ranjeshj
Copy link
Contributor

@carlos-zamora Is the blue color, the default blue shell accent color ?

@ranjeshj ranjeshj added area-Dialogs team-Controls Issue for the Controls team needs-author-feedback Asked author to supply more information. and removed needs-triage Issue needs to be triaged by the area owners labels Apr 30, 2020
@carlos-zamora
Copy link
Member Author

@carlos-zamora Is the blue color, the default blue shell accent color ?

Yes. This one was another one submitted by the accessibility testers. Personally, I feel that using the accent color is correct, but I get how that could create a situation like this where the color contrast is not acceptable.

CC'ing @DHowett-MSFT to keep him in the loop.

@msft-github-bot msft-github-bot added needs-triage Issue needs to be triaged by the area owners and removed needs-author-feedback Asked author to supply more information. labels Apr 30, 2020
@mdtauk
Copy link
Contributor

mdtauk commented Apr 30, 2020

When choosing an Accent Color, it does warn where there could be visibility problems - so if the user has chosen such a colour - then we can presume they take that risk.

The Windows team should have chosen an accent colour that meets contrast ratio recommendations right?

@ranjeshj
Copy link
Contributor

ranjeshj commented May 1, 2020

@chigy can you route this bug ?

@ranjeshj ranjeshj removed the needs-triage Issue needs to be triaged by the area owners label May 1, 2020
@chigy
Copy link
Member

chigy commented May 1, 2020

We are aware of this issue and will be sure to share this as additional information to the team that owns addressing this issue.

@YuliKl
Copy link

YuliKl commented Dec 16, 2020

This issue is now fixed in the OS (by internal bug). Corrected accent color will be the default in the next version of Windows.

@YuliKl YuliKl closed this as completed Dec 16, 2020
@mdtauk
Copy link
Contributor

mdtauk commented Dec 16, 2020

This issue is now fixed in the OS (by internal bug). Corrected accent color will be the default in the next version of Windows.

So this is the end of

00 78 D4

Looking at the design tokens, it looks like the way the Accent colour is applied to Light and Dark themes will be changing. Any chance you could share some information about this new formula for picking colours?

@ghost ghost added the needs-triage Issue needs to be triaged by the area owners label Dec 16, 2020
@StephenLPeters StephenLPeters removed the needs-triage Issue needs to be triaged by the area owners label Dec 17, 2020
@StephenLPeters
Copy link
Contributor

@YuliKl ?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Dialogs team-Controls Issue for the Controls team
Projects
None yet
Development

No branches or pull requests

7 participants