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 Person Card-Style - Custom size properties]: The Luminosity Contrast Ratio of ‘Finance’ text is '3.34:1' which is less than 4.5:1. #3131

Closed
Raisul123 opened this issue Mar 6, 2024 · 1 comment · Fixed by #3176

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-Person-Card’ and activate it.
  4. Navigate to the ‘Style’ and activate it.
  5. Navigate to the ‘Custom size properties’ and activate it.
  6. Check luminosity ratio of ‘Finance’ text with AI tool and observe the issue.

Actual Result:
The Luminosity Contrast Ratio of ‘Finance’ text is '3.34:1' which is less than 4.5:1.

Expected Result:
The Luminosity Contrast Ratio of ‘Finance’ text should be equal or greater than 4.5:1.

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:
mgtp-person-card-CCA fail for Finance text

@vagpt
Copy link
Collaborator

vagpt commented May 13, 2024

This issue is working fine, hence closing this issue.

Snippet:

image

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