microsoftgraph / microsoft-graph-toolkit

Authentication Providers and UI components for Microsoft Graph 🦒
https://docs.microsoft.com/graph/toolkit/overview
Other
945 stars 303 forks source link

{Freeway-Bug}[MGTP-Mgt -agenda -Style -Custom CSS properties’ page]: The Luminosity Contrast Ratio of ‘Tuesday, 27 February, 2024’ Text is '4.3:1' which is less than 4.5:1. #3090

Closed Raisul123 closed 5 months ago

Raisul123 commented 7 months ago

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 ‘Mgt -agenda’ and activate it.
  4. Navigate to the ‘Style’ and activate it.
  5. Navigate to the ‘Custom CSS properties’ page control using tab key and activate it.
  6. Navigate to " Tuesday, 27 February, 2024" text
  7. Check luminosity ration with AI tool and observe the issue.

Actual Result: The Luminosity Contrast Ratio of ‘Tuesday, 27 February, 2024’ Text is '4.3:1' which is less than 4.5:1.

Expected Result: The Luminosity Contrast Ratio of ‘Tuesday, 27 February, 2024’ Text should be greater than or equal to 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: https://www.w3.org/WAI/WCAG21/Understanding/contrast-minimum

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

Attachments:

CCA fail
gavinbarron commented 7 months ago

@Raisul123 There's some strange stuff going on with Accessibility insights here as the color of the text as it is set via CSS is different to what AI is reporting:

CSS color for header text

We'll adjust our colors to address the color contrast issues reported here.

vagpt commented 6 months ago

Hi @gavinbarron,

Could you please let us know in detail about the above comment as we have checked again and getting the same issue.

vagpt commented 5 months ago

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

URL:

https://mgt.dev/next/pr/3179/?path=/story/components-mgt-agenda-style--custom-css-properties

Snippet:

image