-
-
Notifications
You must be signed in to change notification settings - Fork 561
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
fix(icons): changed shopping-bag
icon
#2483
base: main
Are you sure you want to change the base?
Conversation
Added or changed iconsIcons as codeWorks for: const ShoppingBagIcon = createLucideIcon('ShoppingBag', [
["path",{"d":"M16 10a4 4 0 0 1-8 0"}],
["path",{"d":"M3.103 6.034h17.794"}],
["path",{"d":"M3.4 5.467a2 2 0 0 0-.4 1.2V20a2 2 0 0 0 2 2h14a2 2 0 0 0 2-2V6.667a2 2 0 0 0-.4-1.2l-2-2.667A2 2 0 0 0 17 2H7a2 2 0 0 0-1.6.8z"}]
]) |
Throwing this complete redesign in here, tbh, I'm not a fan of our current shopping bag icon: |
The current version is also a bit big, I don't like it when it is smaller tho, I feel like it needs to be taller than it is wide. |
I kind agree, which is exactly why the one I posted – while the base shape is almost square – I think has a kind of "illusion of narrowness" because of the extra handle. |
I'm up for it, but maybe we should merge this and create a separate PR for it. |
This PR is stale because it has been open 45 days with no activity. Remove stale label or comment or this will be closed in 5 days. |
What is the purpose of this pull request?
Description
Arcified icon.
Before Submitting