-
-
Notifications
You must be signed in to change notification settings - Fork 685
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
add scale icon #205
add scale icon #205
Conversation
mittalyashu
commented
Feb 3, 2021
Preview |
---|
![]() |
@mittalyashu Hmm I think we can refine this icon some more. |
Sure |
@mittalyashu |
@ericfennis can you add 1px more spacing here
|
@mittalyashu Like this? |
@ericfennis The spacing looks fine to me, what about adding a curvy slant on both left and right side? |
@mittalyashu Mneeh I'm not really fan of it, I think we should keep it simple. |
@ericfennis thats not what I mean. Can you provide the source of the icon? |
@mittalyashu Yes of course! <svg width="24" height="24" viewBox="0 0 24 24" fill="none" xmlns="http://www.w3.org/2000/svg">
<path d="M16 16L19 8L22.001 16C21.1353 16.649 20.0825 16.9999 19.0005 16.9999C17.9185 16.9999 16.8657 16.649 16 16Z" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M2 16L5 8L8.001 16C7.1353 16.649 6.08248 16.9999 5.0005 16.9999C3.91852 16.9999 2.8657 16.649 2 16Z" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M7 21H17" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M12 3V21" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
<path d="M3 7H7L12 5L17 7H21" stroke="black" stroke-width="2" stroke-linecap="round" stroke-linejoin="round"/>
</svg> |
@mittalyashu I think the icons looks better without slant. |
@ericfennis That's what I mean by adding curves. What do you think? |
@mittalyashu looks great, I think this is the one. |
@mittalyashu is attempting to deploy a commit to the Lucide Team on Vercel. A member of the Team first needs to authorize it. |
@ericfennis LGTM |
This pull request is being automatically deployed with Vercel (learn more). 🔍 Inspect: https://vercel.com/lucide/lucide/FUCjv2ukd3YoRY7R6Ezqn6NjTyJe |
Looks good! |