-
-
Notifications
You must be signed in to change notification settings - Fork 572
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(icons): adds barrel
icon
#1955
base: main
Are you sure you want to change the base?
Conversation
Added or changed iconsIcons as codeWorks for: const BarrelIcon = createLucideIcon('Barrel', [
["path",{"d":"M16 20.5v-17"}],
["path",{"d":"M8 3.5v17"}],
["path",{"d":"M2 10a50.5 50.5 0 0 1 20 0"}],
["path",{"d":"M2 14a50.5 50.5 0 0 0 20 0"}],
["path",{"d":"M21.16 5.677A17 17 0 0 0 12 3a17 17 0 0 0-9.16 2.677A2 2 0 0 0 2 7.268v9.464a2 2 0 0 0 .84 1.591A17 17 0 0 0 12 21a17 17 0 0 0 9.16-2.677 2 2 0 0 0 .84-1.591V7.268a2 2 0 0 0-.84-1.591"}]
]) |
Feels more like a skewed table than a barrel to me. |
I don't see the barrel as well. Maybe something based on the database icon? |
Maybe giving it a tap and some feet can help? Another icon: |
@karsa-mistmere What about this design? |
I explored something similar but it's just too dense this way. |
I think it is not that bad |
Well, in practice the corners do not fulfil the 2px gap rule, I'm afraid. 😅 |
Do you still have that isomorphic view flying around somewhere? |
Maybe moving the rings around the barrel more towards the edges helps? And add a tap for good measure. Or still too dense? |
Even without the tap, your version looks quite barrel-y. |
What is the purpose of this pull request?
Description
Icon use case
Icon Design Checklist
Concept
Author, credits & license
Naming
icons/[iconName].json
.Design
Before Submitting