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

fix(data-table): high contrast mode fixes #6873

Merged
merged 3 commits into from
Sep 22, 2020

Conversation

tw15egan
Copy link
Collaborator

@tw15egan tw15egan commented Sep 21, 2020

Closes #6607
Closes #6871

Adds in a specific selector for Windows HCM mode to enable proper SVG fill colors.

CSS Tricks article about accessible SVGs in HCM

Changelog

New

  • Specific Windows HCM media queries to use the CSS2 system color ButtonText as the fill property for our SVG icons

Testing / Reviewing

On Windows, go to Settings --> Ease of Access --> High Contrast, and enable High Contrast mode. Ensure all icons are visible in all Data Table variants. Experience in HCM should be the same as a user not in HCM.

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-elements ready!

Built with commit 8512481

https://deploy-preview-6873--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-components-react ready!

Built with commit 8512481

https://deploy-preview-6873--carbon-components-react.netlify.app

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-elements ready!

Built with commit 5ef3c2e

https://deploy-preview-6873--carbon-elements.netlify.app

@netlify
Copy link

netlify bot commented Sep 21, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 5ef3c2e

https://deploy-preview-6873--carbon-components-react.netlify.app

@joshblack
Copy link
Contributor

bump @andreancardona when you get a sec today

Copy link
Contributor

@andreancardona andreancardona left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@tw15egan looks good to merge :)

@kodiakhq kodiakhq bot merged commit 31dab58 into carbon-design-system:master Sep 22, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants