-
Notifications
You must be signed in to change notification settings - Fork 30
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
docs(Icon): updating documentation #2881
Conversation
@@ -26,6 +26,14 @@ import { | |||
args={{}} | |||
/> | |||
|
|||
## General Infomation | |||
|
|||
For mini icons, we use a subset of [Streamline's "Core Solid" icons](https://www.streamlinehq.com/icons/core-solid). |
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.
Check with Stacey if this link is ok
@@ -46,25 +46,12 @@ import { | |||
|
|||
## General Infomation | |||
|
|||
We use a subset of [streamline icons](https://app.streamlineicons.com/streamline-regular), mainly the Outline version of the Regular set. | |||
For regular icons, we use a subset of [Streamline's "Ultimate Regular" icons](https://www.streamlinehq.com/icons/streamline-regular). |
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.
Check with Stacey if this link is ok (and removal of the clause is ok) seems like Regular don't use Outline? or shouldn't?
📬Published Alpha Packages:@codecademy/[email protected] |
🚀 Styleguide deploy preview ready! |
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.
LGTM!
Overview
Reorganizes documentation for icon creation in storybook
Updating Icon documentation
PR Checklist
Testing Instructions
Don't make me tap the sign.