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

feat(Button): add as prop to IconButton inside of the Button component #12900

Merged

Conversation

matkrzy
Copy link
Contributor

@matkrzy matkrzy commented Dec 23, 2022

Button with flag hasIconOnly renders Button component without as prop so we are not able to pass custom render component

Changelog

Changed

  • IconButton inside of the Button component has new prop as

Testing / Reviewing

  1. set flag hasIconOnly
  2. pass custom component via as property
  3. custom component should render instead default button

@matkrzy matkrzy requested a review from a team as a code owner December 23, 2022 12:34
@github-actions
Copy link
Contributor

github-actions bot commented Dec 23, 2022

DCO Assistant Lite bot All contributors have signed the DCO.

@netlify
Copy link

netlify bot commented Dec 23, 2022

Deploy Preview for carbon-components-react ready!

Built without sensitive environment variables

Name Link
🔨 Latest commit cead11b
🔍 Latest deploy log https://app.netlify.com/sites/carbon-components-react/deploys/63bda6d37eff2200081512dd
😎 Deploy Preview https://deploy-preview-12900--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.

@netlify
Copy link

netlify bot commented Dec 23, 2022

Deploy Preview for carbon-elements ready!

Name Link
🔨 Latest commit cead11b
🔍 Latest deploy log https://app.netlify.com/sites/carbon-elements/deploys/63bda6d265336c0009719d79
😎 Deploy Preview https://deploy-preview-12900--carbon-elements.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.

@matkrzy matkrzy force-pushed the feature/button-icon-as-property branch from 564abc7 to 74d9c2a Compare December 23, 2022 12:46
@matkrzy
Copy link
Contributor Author

matkrzy commented Dec 23, 2022

I have read the DCO document and I hereby sign the DCO.

@matkrzy matkrzy force-pushed the feature/button-icon-as-property branch from 74d9c2a to 869c65b Compare December 23, 2022 12:51
Copy link
Member

@tay1orjones tay1orjones left a comment

Choose a reason for hiding this comment

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

Great catch, thanks!

@matkrzy matkrzy force-pushed the feature/button-icon-as-property branch from 869c65b to 4724b09 Compare January 10, 2023 08:04
@kodiakhq kodiakhq bot merged commit 869d163 into carbon-design-system:main Jan 10, 2023
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.

3 participants