Skip to content
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

add package plus, minus, x, search, check icons #474

Merged
merged 15 commits into from
Feb 22, 2022
Merged

add package plus, minus, x, search, check icons #474

merged 15 commits into from
Feb 22, 2022

Conversation

Reund0
Copy link
Contributor

@Reund0 Reund0 commented Jan 14, 2022

adding package group icons including:

package-plus icon,

package-plus

package-minus icon,

package-minus

package-search icon,

package-search

package-x icon,

package-x

package-check icon.

package-check

Copy link
Member

@ericfennis ericfennis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Reund0 Nice icons! These are really welcome.

Did you read the Lucide Icon Design guide?
We like to see 1px padding of the 24x24 pixel grid. I see there is some place to move the icon a bit to the right to center it.

image
Also maybe you can try to increase the sub-icons in the corner a bit if it fits.
If you need any help let me know.

Other things we like to see is that the icon if formatted like all the other icons. See Global Attributes in icon design guide, or look a an other icon in the icons directory.

And at least please add some tags to the tags.json located in the root directory. This improves the search engine on the website. So other devs and designers will find your beautiful icons!

@mittalyashu
Copy link
Member

The package search icon seems a bit congested.

@Reund0 Can you try increase the size of circle of magnifying glass?

@Reund0
Copy link
Contributor Author

Reund0 commented Jan 21, 2022

The package search icon seems a bit congested.

@Reund0 Can you try increase the size of circle of magnifying glass?

Is it okay to not have 2px spacing?

@mittalyashu
Copy link
Member

mittalyashu commented Jan 23, 2022

@Reund0

Is it okay to not have 2px spacing?

Ah! I see, what you mean. That will be bit tricky 🤔

@ericfennis Do you think it would be good idea to keep the package icon edge to edge that way the children icon (not just search other icons as well) will get more space.

@ericfennis
Copy link
Member

@mittalyashu I think it's better to leave it centered, that matches better it the current package icon if used together.

@ericfennis
Copy link
Member

@Reund0 Can you also increase the size of the X a bit in the package-x icon?

@ericfennis ericfennis added the 🎨 icon About new icons label Feb 1, 2022
changed from 4x4 to 4.5x4.5
@Reund0
Copy link
Contributor Author

Reund0 commented Feb 4, 2022

@Reund0 Can you also increase the size of the X a bit in the package-x icon?

I increased its size from 4x4 to 4.5x4.5 but again is it okay to ignore 2px spacing?

Copy link
Member

@ericfennis ericfennis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The svgs shouldn't contain any "combined" paths. This renders not well in design programs like figma.
Screenshot:
image

I added some suggestion to fix it!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎨 icon About new icons
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants