-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
feat(Search): customize icon #8537
feat(Search): customize icon #8537
Conversation
Deploy preview for carbon-elements ready! Built with commit ada0e93 |
Deploy preview for carbon-components-react ready! Built without sensitive environment variables with commit ada0e93 https://deploy-preview-8537--carbon-components-react.netlify.app |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @FabianPiconeDev - I added a commit here to include the PropTypes definition as well as rename the prop to renderIcon
to be more consistent with other components that accept an icon as a prop.
@tw15egan Sorry I was a bit busy, I was just about to do it, then I saw that @tay1orjones did it. Thanks! :-) |
Closes #8480