-
-
Notifications
You must be signed in to change notification settings - Fork 701
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 scan-document
icon
#2882
base: main
Are you sure you want to change the base?
Conversation
Metadata for `scan-document.svg`
Added or changed iconsIcons as codeWorks for: const ScanDocumentIcon = createLucideIcon('ScanDocument', [
["path",{"d":"M2 18V22H6M2 6V2H6M22 6V2H18M18 22H22V18M18 10V17C18 18.1046 17.1046 19 16 19H8C6.89543 19 6 18.1046 6 17V7C6 5.89543 6.89543 5 8 5H13M18 10H15C13.8954 10 13 9.10457 13 8V5M18 10V9L14 5H13M9 16H15M15 13H9M9 10H10"}]
]) |
There is already a Irrespective of that, there are a couple issue with the icon. The spacing is too tight in multiple spots. There needs to be 2 px spacing around the strokes. If you were to properly space out the icon, you wouldn't be able to fit the "file text lines" in the icon. Also, the scanner part of the icon isn't in line with the other I created a version that has proper spacing and consistency with the other icons: As you can see, it's pretty tight so you cant really fit anything in the file icon. |
The Overall, your version has a right to exist 👍 |
What is the purpose of this pull request?
Description
Adds a
scan-document
iconIcon use case
This icon can be useful as a visual representation of a document or file being scanned
Icon Design Checklist
Concept
Author, credits & license
scan
&file-text
Naming
icons/[iconName].json
.Design
Before Submitting