-
-
Notifications
You must be signed in to change notification settings - Fork 917
feat: add naira-sign icon #3480
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
base: main
Are you sure you want to change the base?
Conversation
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.
🤖 ChatGPT Tags suggestions ✨
I've asked ChatGPT for some suggestions for tags.
Added or changed iconsIcons as codeWorks for: const NairaSignIcon = createLucideIcon('NairaSign', [
["path",{"d":"M6 4V20"}],
["path",{"d":"M18 4V20"}],
["path",{"d":"M4 8H20"}],
["path",{"d":"M4 16H20"}],
["path",{"d":"M6 4L18 20"}]
]) |
Has gap guideline violations in the corners. |
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
I have corrected the gap violation and updated with the correct corners. |
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.
Visual size is low, and the 2px gap violation is still there.
This barely adds a 2px gap into the corners: |
Other possible resolutions for the gap issue: |
@jguddas I just made another correction now. Please let me know if it passes or if I need to make any further changes; I'm happy to do so. Thank you for your comment. |
Description
This PR introduces a new icon: naira-sign, representing the official currency symbol of Nigeria (₦).
Icon use case
Alternative icon designs
No alternatives at this time. Open to feedback and improvements.
Icon Design Checklist
Concept
Author, credits & license
Naming
icons/naira-sign.json.
.Design
Before Submitting