-
-
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): added shield-user
icon
#2608
base: main
Are you sure you want to change the base?
Conversation
Added or changed iconsIcons as codeWorks for: const ShieldUserIcon = createLucideIcon('ShieldUser', [
["path",{"d":"M20 13c0 5-3.5 7.5-7.66 8.95a1 1 0 0 1-.67-.01C7.5 20.5 4 18 4 13V6a1 1 0 0 1 1-1c2 0 4.5-1.2 6.24-2.72a1.17 1.17 0 0 1 1.52 0C14.51 3.81 17 5 19 5a1 1 0 0 1 1 1z"}],
["path",{"d":"M6.376 18.91a6 6 0 0 1 11.249.003"}],
["circle",{"cx":"12","cy":"11","r":"4"}]
]) |
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. |
I would prefer something like this |
Feels a bit imbalanced, maybe a smaller user would be more appropriate: |
My version is from the |
It's not entirely consistent though, since the user shape is 1px lower than in case of I feel like even with the minor gap guideline violation this one feels better because it's much less bottom-heavy: |
I'm up for this, even with the gap violation, looks nice. |
Thanks guys for the contribution, I also like this version, changing now |
Apply suggestion by @karsa-mistmere at lucide-icons#2608 (comment)
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 think I have managed to do that, please check: |
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.
Yeah, sorry, my bad, the one I posted was just a rough draft.
This new version LGTM!
The three splines should be a single arc |
I'm sorry, but my SVG editing skills have reached a limit with this one, I can't create a pixel-perfect arc there. Can you help me fix it, please? |
Thank you for the detailed answer! I will remember this. I commited the modified svg. |
closes #2600
What is the purpose of this pull request?
Description
Added new
shield-user
icon.Icon use case
Access to admin area, mark user as admin/elevated access rights
Alternative icon designs
Icon Design Checklist
Concept
Author, credits & license
shield
,user
Naming
icons/[iconName].json
.Design
Before Submitting