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

Added Gumroad filled icon and corrected file location in README #219

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Conversation

flashblaze
Copy link

I have added filled SVG icon for Gumroad and edited the corresponding digest file. Do let me know whether I have made any mistakes while adding an icon or regarding any changes.

Thanks.

@vagusX vagusX requested a review from AntDesigners January 5, 2020 08:24
@vagusX
Copy link
Member

vagusX commented Jan 5, 2020

Maybe you could use pass the svg to Icon component directly

@flashblaze
Copy link
Author

Do you mean I should run

npm run generate
npm run compile
npm publish

to create the Icon components?

@vagusX
Copy link
Member

vagusX commented Jan 7, 2020

@flashblaze
Copy link
Author

@vagusX thanks for the link.

I wanted to contribute some icons here. Is there any way I can do so?

@vagusX
Copy link
Member

vagusX commented Jan 9, 2020

Maybe you can have a look at #71?

@flashblaze
Copy link
Author

Thanks

@flashblaze flashblaze mentioned this pull request Jan 9, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants