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(v10): backport HCM fixes to v10 for RadioButton, Toggle, Notification #12100

Merged
merged 3 commits into from
Sep 23, 2022

Conversation

tw15egan
Copy link
Collaborator

Closes #11648
Closes #10458
Closes #10199
Refs #12098
Refs #12097

This backports a few V11 fixes that affected RadioButton, Toggle and Notification when using Windows HCM

Changelog

Changed

  • Added/Updated HCM selectors on RadioButton, Toggle and Notification

Testing / Reviewing

In Windows, enable a Dark High Contrast mode theme, and check out RadioButton, Toggle and Notification. Also verify that the icons show up in a light theme (Desert if on Windows 11). Make sure the components are completely visible and no elements are missing

@tw15egan tw15egan requested a review from a team as a code owner September 15, 2022 15:44
@tw15egan tw15egan requested review from tay1orjones and aledavila and removed request for a team September 15, 2022 15:44
@tw15egan tw15egan changed the title fix(v10): backport HCM fixes to v10 for RadioButton, Toggle fix(v10): backport HCM fixes to v10 for RadioButton, Toggle, Notification Sep 15, 2022
@netlify
Copy link

netlify bot commented Sep 15, 2022

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit 85b4532
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/632dcfce6779c50009ee2711
😎 Deploy Preview https://deploy-preview-12100--carbon-components-react.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site settings.

@tay1orjones tay1orjones requested review from abbeyhrt and removed request for tay1orjones September 15, 2022 16:42
Copy link
Contributor

@abbeyhrt abbeyhrt left a comment

Choose a reason for hiding this comment

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

Looks good to me! Tested it out in HCM black and white and everything is visible!

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

Successfully merging this pull request may close these issues.

3 participants