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

A11y_.NET Core_ASP.NETCore_WebAppMVCRegister_NonTextContrast: The Color contrast ratio of the focus indicator of the "Email", "Password", "Confirm password" and "Register" button is less than 3:1. #42397

Closed
Swati1700 opened this issue Jun 24, 2022 · 7 comments
Assignees
Labels
area-identity Includes: Identity and providers feature-accessibility This issue has accessibility impact
Milestone

Comments

@Swati1700
Copy link

Swati1700 commented Jun 24, 2022

Check out Accessibility Insights! - Identify accessibility bugs before check-in and make bug fixing faster and easier.”

GitHubTags:#A11yMAS;#A11yWCAG2.1;#A11yTCS;#.NET Core[aspnetcore]-Win32-June2022;#.NET Core;#A11ySev2;#WCAG1.4.11;#DesktopWeb;#Win11;#ColorContrast;#Benchmark;#Chrome

Environment Details:

.NET Core
ASP.NET Core
Operating System: Windows 11 Enterprise 21H2
Chrome: Version 102.0.5005.115 (Official Build) (64-bit)

Steps to Reproduce:

  1. open command prompt
  2. Create a folder using command "mkdir foldername" e.g. "mkdir MVC"
  3. move to the created directory using command "cd foldername"
  4. run the following command "dotnet new mvc -au Individual" and hit enter.
  5. run the following command "dotnet run"
  6. Copy the First URL link from Command prompt and copy it
  7. Paste it in URL section of Browser and hit Search
  8. browser page will open with above template loaded
  9. Test the Register page
  10. Turn on Color contrast analyzer and check that The Color contrast ratio of focus indicator for "Email", "Password" "Confirm Password" and "Register" is less than 3:1.

Actual:

The Color contrast ratio of the focus indicator of the "Email", "Password", "Confirm password" and "Register" button is less than 3:1.

Similar issue also observed in the below scenarios:
Issue 1: ASP.NET Core - WebAppMVC Login

  1. open command prompt
  2. Create a folder using command "mkdir foldername" e.g. "mkdir MVC"
  3. move to the created directory using command "cd foldername"
  4. run the following command "dotnet new mvc -au Individual" and hit enter.
  5. run the following command "dotnet run"
  6. Copy the First URL link from Command prompt and copy it
  7. Paste it in URL section of Browser and hit Search
  8. browser page will open with above template loaded
  9. Register and then hit enter on Login page
  10. test login page
  11. Turn on Color contrast analyzer and check that The Color contrast ratio of focus indicator for "Email", "Password", "Remember me?" and "Log in" button is less than 3:1.

Issue2: ASP.NET Core - WebAppMVC Forgot Password

  1. open command prompt
  2. Create a folder using command "mkdir foldername" e.g. "mkdir MVC"
  3. move to the created directory using command "cd foldername"
  4. run the following command "dotnet new mvc -au Individual" and hit enter.
  5. run the following command "dotnet run"
  6. Copy the First URL link from Command prompt and copy it
  7. Paste it in URL section of Browser and hit Search
  8. browser page will open with above template loaded
  9. Register and then hit enter on Login page
  10. Tab and Navigate to "Forgot your password?" link and Hit Enter.
  11. Turn on Color contrast analyzer and check that The Color contrast ratio of focus indicator for "Email", "Reset Password" button is less than 3:1.

Expected:

The Color contrast ratio of the focus indicator of the "Email", "Password", "Confirm password" and "Register" button should be more than or equal to 3:1.

User Impact:

Low vision users won't be able to see the focus indicator.

Screenshot (1136)

@javiercn javiercn added area-blazor Includes: Blazor, Razor Components feature-accessibility This issue has accessibility impact labels Jun 24, 2022
@javiercn javiercn added this to the 7.0-preview7 milestone Jun 24, 2022
@mkArtakMSFT mkArtakMSFT modified the milestones: 7.0-preview7, 7.0-rc1 Jul 13, 2022
@MackinnonBuck MackinnonBuck added area-identity Includes: Identity and providers and removed area-blazor Includes: Blazor, Razor Components labels Aug 3, 2022
@MackinnonBuck MackinnonBuck assigned HaoK and unassigned MackinnonBuck Aug 3, 2022
@HaoK HaoK assigned Swati1700 and unassigned HaoK Aug 10, 2022
@HaoK
Copy link
Member

HaoK commented Aug 10, 2022

Should be fixed by #43177

@Swati1700
Copy link
Author

Swati1700 commented Aug 19, 2022

@HaoK We downloaded the latest RC1 build from the first column of the above table and verified the bug, and the bug still reproduces.
Please find the attachment of the version.
image
Could you please provide the repro steps that you followed, if the issue is not reproducing at your end.

@merriemcgaw
Copy link
Member

@Swati1700 - latest builds should now have it. There was a small issue with flowing into the installers, but it should be addressed now.

@Swati1700
Copy link
Author

We have installed latest build dotnet-sdk-7.0.100-rc.2.22452.3-win-x64 but getting a below error.
image

@merriemcgaw
Copy link
Member

@Swati1700 - the updated machine setup instructions are as follows:

  1. On a clean machine install .NET RC2 SDK from the Top, Middle cell in the Installer Table
  2. Open a command prompt and enter the following command: dotnet nuget add source -n dotnet7 https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet7/nuget/v3/index.json
  3. Follow the test repro steps from here.

@Swati1700
Copy link
Author

Verified the issue and the issue is fixed. Hence closing the bug.
image

@ghost ghost locked as resolved and limited conversation to collaborators Oct 27, 2022
@mkArtakMSFT
Copy link
Member

Issue is fixed. Adding #Closed; tag.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-identity Includes: Identity and providers feature-accessibility This issue has accessibility impact
Projects
None yet
Development

No branches or pull requests

6 participants