-
Notifications
You must be signed in to change notification settings - Fork 114
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(icons): add CopyIcon #235
Conversation
This pull request is being automatically deployed with ZEIT Now (learn more). 🔍 Inspect: https://zeit.co/twilio-dsys/paste/6nuteh0x0 |
import {LoadingIcon} from './src/LoadingIcon'; | ||
import {PlusIcon} from './src/PlusIcon'; | ||
|
||
interface IconProps { | ||
title?: string; | ||
decorative?: boolean; |
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.
I followed the instructions outlined here: https://github.com/twilio-labs/paste/blob/master/packages/paste-icons/README.md#how-can-i-contribute-icons
Interesting the tool removed the ?
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.
Yup this change is good. The ?
there was an oversight.
@@ -0,0 +1,43 @@ | |||
/** | |||
* This file was automatically generated with @twilio-labs/svg-to-react |
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.
This was very smooth!
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.
Thanks! :D
Hey @hharnisc! I'm going to send you an updated SVG tonight. We're going to make the square in the icon more square. Thanks for being proactive on this! |
Sounds great @serifluous 🙌 I'll update the PR with the new SVG |
@TheSisb I updated the |
Adds the CopyIcon with the same SVG as the internal react component library
Contributing to Twilio