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

chore(Toggle): changing fill color for disabled but checked small toggle #6955

Merged

Conversation

luizamendes
Copy link
Contributor

Closes #6659

The objective is to change the style of the small toggle when it is checked but disabled.
This is my first time contributing ever.

Changelog

Changed

  • Changed the ToggleSmall component, in the checked but disabled state.
  • Changed the fill of the check icon from $disable-02 to $disable-01, what makes the check icon appear.

Testing / Reviewing

Start the storybook -> from packages/react folder yarn storybook
Go to ToggleSmall component -> checked version
Go to knobs and set it as disabled
You should see the check icon, stating that although the toggle is disabled it is checked.

@luizamendes luizamendes requested a review from a team as a code owner October 1, 2020 20:53
@netlify
Copy link

netlify bot commented Oct 1, 2020

Deploy preview for carbon-elements ready!

Built with commit dbec95a

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

@netlify
Copy link

netlify bot commented Oct 1, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit dbec95a

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

@netlify
Copy link

netlify bot commented Oct 1, 2020

Deploy preview for carbon-elements ready!

Built with commit 309b90c

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

@netlify
Copy link

netlify bot commented Oct 1, 2020

Deploy preview for carbon-components-react ready!

Built without sensitive environment variables with commit 309b90c

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

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.

@luizamendes this is a great first contribution! 🎊 Getting visual approval - but looks good on my end!

@aagonzales let me know if this looks good to you! See screenshots:

After change:
Screen Shot 2020-10-02 at 8 51 05 AM

Before change:
Screen Shot 2020-10-02 at 8 51 17 AM

Copy link
Member

@aagonzales aagonzales left a comment

Choose a reason for hiding this comment

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

This looks great! Thanks for contributing!

Copy link
Collaborator

@tw15egan tw15egan left a comment

Choose a reason for hiding this comment

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

🎉 Congrats @luizamendes on your first contribution! Looks great 👍 ✅

@kodiakhq kodiakhq bot merged commit fded63c into carbon-design-system:master Oct 2, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Toggle on-disabled just looks disabled
4 participants