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

[Bug]: Luminosity contrast ratio of keyboard focus indicator on 'Left navigation' controls is 2.604:1 which is less than 3:1 #24162

Closed
msftedad opened this issue Sep 13, 2023 · 2 comments · Fixed by #24205

Comments

@msftedad
Copy link

Describe the bug

Luminosity contrast ratio of keyboard focus indicator on 'Left navigation' controls is 2.604:1 which is less than 3:1

Actual Result:
The color contrast ratio of the keyboard focus indicator for the 'Left navigation' controls is 1.252:1 which is less than 3:1

Expected Result:
The color contrast ratio of the keyboard focus indicator on the 'Left navigation controls' should be greater than or equal to 3:1.

To Reproduce

  1. Open the URL- Samples / General - Localization ⋅ Storybook (mgt.dev)
  2. 'Microsoft Graph Toolkit Playground' Overview page will get displayed.
  3. Navigate to the 'Left rail navigation' region using tab key.
  4. Navigate on the controls using tab key.
  5. Run AI tool and observe the issue.

Attachments:
ScreenShot

System

OS Build: Windows 11 
Version: 22H2 (OS Build 22621.2134) 
Browser: Edge dev 
Browser Version 116.0.1938.76 (Official build) (64-bit)

Additional context

Same issue is repro on Components / mgt-agenda / Properties - Preferred Timezone ⋅ Storybook

@chocoscoding
Copy link
Contributor

chocoscoding commented Sep 13, 2023

@shilman i would like fix this, can i be assigned

@shilman
Copy link
Member

shilman commented Sep 14, 2023

@chocoscoding Awesome! Thank you for helping make Storybook better! 🙏 Please check out how to contribute in our docs and feel free to ask questions in #contributing in our Discord.

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

Successfully merging a pull request may close this issue.

3 participants