-
Notifications
You must be signed in to change notification settings - Fork 2
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/144 oss icons #176
Feat/144 oss icons #176
Conversation
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.
Looks good! No comments.
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.
Looks good! No comments.
@swdilip Some icons are meant to be bigger than others, hence the difference in thickness. For dark mode, we allow the user to pass in props to the icons, so that they may alter the colors directly :D |
56111ee
to
0c321f8
Compare
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.
Looks good to me! LGTM!
Fixes #144
Also I added an mdx file to list out all the icons, renamed every single icon to be more clear, moved over the index file into components so our src index file can be more concise