-
Notifications
You must be signed in to change notification settings - Fork 80
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: add arrow upward icon #2079
Conversation
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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 like the build is failing because of naming issues. We also have to add the new exports to packages/components/icons/src/index.ts
Co-authored-by: Kristoffer <denkristoffer@users.noreply.github.com>
Co-authored-by: Kristoffer <denkristoffer@users.noreply.github.com>
The SVG itself looks off, are the paths correct? https://5fd1dda724cc620021ace8c5-agzeekenuo.chromatic.com/?path=%2Fstory%2Fcomponents-icons--overview |
</Fragment> | ||
), | ||
trimmed: true, | ||
viewBox: '0 0 48 48', |
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.
If the view box is the same as the "regular" icon we don't need the trimmed variant.
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.
What is the "regular" icon size?
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.
Oh, you mean the size defined by material design package. Ok, removing it
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 mean the ArrowUpward.tsx
you're adding. Trimmed is for cases where you have an icon with a view box smaller than 24x24 and you want a variant without the extra padding. I know this is not clear from the docs 🙃
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.
Great, thanks!
* feat: add arrow upward icon Co-authored-by: Kristoffer <denkristoffer@users.noreply.github.com>
Purpose of PR
PR Checklist
readme.md
file(s)