-
Notifications
You must be signed in to change notification settings - Fork 841
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] Add link to Figma UI Kit #3016
Conversation
Preview documentation changes for this PR: https://eui.elastic.co/pr_3016/ |
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.
Nice! I like that you changed "Sketch" to just "Libraries". You might want to add a tooltip around the Figma one as well for those unfamiliar with the logo.
Preview documentation changes for this PR: https://eui.elastic.co/pr_3016/ |
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 for adding this. Just had one extra comment for the old Sketch link
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.
Same comment as @cchaos. Thanks for adding this quick. Good to merge once changes are in.
cc @hbharding so he's aware there will be a more prominent link. |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3016/ |
Preview documentation changes for this PR: https://eui.elastic.co/pr_3016/ |
Thanks @miukimiu ! |
Summary
Closes #3014.
This PR adds a link to the Figma UI Kit into the home page.
The Figma icon was optimized and resized to the same size as other third-parties icons. It was used as a custom SVG as mentioned on #3014.
Checklist
[ ] Props have proper autodocs[ ] Added documentation examples[ ] Added or updated jest tests[ ] Checked for breaking changes and labeled appropriately[ ] A changelog entry exists and is marked appropriately