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

[MGTP-People-picker- Style- Custom size properties]: The Luminosity Contrast Ratio of ‘Search for a name’ placeholder text of search edit field is '3.686:1' which is less than 4.5:1. #3128

Closed
Raisul123 opened this issue Mar 5, 2024 · 1 comment · Fixed by #3175

Comments

@Raisul123
Copy link
Collaborator

Test Environment:
OS Build: Windows 11 
Version: 24H2 (OS Build 26058.1400) 
Browser: Edge dev 
Browser Version 123.0.2400.1 (Official build) dev (64-bit)
URL: Overview - Docs ⋅ Storybook (mgt.dev)
Tool: Accessibility Insight for web

Repro Steps:

  1. Open the above URL.
  2. The ‘Microsoft Graph Toolkit Playground' Overview page will get displayed.
  3. Navigate to the ‘MGTP-People-picker’ and activate it.
  4. Navigate to the ‘Style’ and activate it.
  5. Navigate to the ‘Custom size properties’ and activate it.
  6. Navigate to the ‘Search for a name’ placeholder text of search edit
  7. Check luminosity ratio with AI tool and observe the issue.

Actual Result:
The Luminosity Contrast Ratio of ‘Search for a name’ placeholder text of search edit field is '3.686:1' which is less than 4.5:1.

Expected Result:
The Luminosity Contrast Ratio of ‘Search for a name’ placeholder text of search edit field should be equal or greater than 4.5:1.

Note:
Also Luminosity Contrast Ratio of ‘We didn’t find any matches’ text which is appeared when focus land on search edit field is '4.351:1' which is less than 4.5:1. (PFA)

User Impact:
Low vision users would face difficulty in seeing the text if the color contrast ratio of the control is less than 4.5:1.

WCAG Reference:
Understanding Success Criterion 1.4.3: Contrast (Minimum) | WAI | W3C

“ Have feedback on bugs logged, please tag bug as “A11yRCA“ and add your feedback in the comment section. “

Attachments:
CCA fail for search for a name text
Note CCA fail for we did not fined any matches text

@vagpt
Copy link
Collaborator

vagpt commented May 13, 2024

This issue is working fine on the URL below, hence closing this issue.

URL:
https://mgt.dev/next/pr/3175/?path=/story/components-mgt-people-picker-styles--custom-css-properties

Snippet:

image

image

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment