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(structured-list): remove icon from hover state #8734

Conversation

emyarod
Copy link
Member

@emyarod emyarod commented May 21, 2021

Closes #8733

This PR removes the checkmark icon from the structured list rows on hover

@netlify
Copy link

netlify bot commented May 21, 2021

Deploy Preview for carbon-elements ready!

Built with commit 9b0d12c

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

@netlify
Copy link

netlify bot commented May 21, 2021

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables with commit 9b0d12c

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

Copy link
Member

@kingtraceyj kingtraceyj left a comment

Choose a reason for hiding this comment

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

thanks andrew. looks good! 💥

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.

🎉

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.

LGTM 👍 ✅

@kodiakhq kodiakhq bot merged commit b9090b0 into carbon-design-system:main May 24, 2021
@jnm2377 jnm2377 mentioned this pull request May 24, 2021
22 tasks
@emyarod emyarod deleted the 8733-structuredlist-remove-icon-from-hover-state branch May 25, 2021 15:05
@emyarod emyarod mentioned this pull request Jun 1, 2021
87 tasks
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.

[structured list] Remove icon from hover state
4 participants