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.

Sign up for GitHub

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

Jump to bottom

Add brush icon #108

Merged
merged 9 commits into from Dec 30, 2020
Merged

Add brush icon #108

merged 9 commits into from Dec 30, 2020

Conversation

ghost
Copy link

@ghost ghost commented Oct 22, 2020 •
edited by ghost
Loading

Hello! I created this icon.
image
100%
brush
There are too many commits because I wanted to keep my fork up to date

@ghost
Copy link
Author

ghost commented Oct 23, 2020

Any feedback?

@ericfennis
Copy link
Member

@delnyn Nice work.
Can you try to make an version with a perfectly diagonal stick?
And one with the brush facing down, just to compare.

@ericfennis ericfennis added the 🎨 icon About new icons label Oct 25, 2020
@ghost
Copy link
Author

ghost commented Oct 28, 2020 •
edited by ghost
Loading

There you go @ericfennis
image

@ericfennis
Copy link
Member

I think like it more with the tip down.
It would be nice if the icon matches with the body of the edit-2 icon.

@ghost
Copy link
Author

ghost commented Oct 31, 2020

yup no problem

@ghost
Copy link
Author

ghost commented Nov 11, 2020

Is it good?
image

@ericfennis
Copy link
Member

@delnyn I think maybe more like this. (Material icon)
image
The stick is good. I think we can separate the brush from the stick.
And I think the brush can be more "wipey".

@ghost
Copy link
Author

ghost commented Nov 15, 2020 •
edited by ghost
Loading

Is it good?
image
100%
brush

@ericfennis
Copy link
Member

@delnyn Nice this one looks awesome.
Can you maybe try one thing, maybe it looks a bit cleaner removing this stroke.

image

@ghost
Copy link
Author

ghost commented Nov 19, 2020

Okay!

@ericfennis
Copy link
Member

@delnyn Any progress on this?
Let me know if you need any help!

@ghost
Copy link
Author

ghost commented Dec 20, 2020

Oh yes, sorry! I'm having trouble exporting the SVG...

@ghost
Copy link
Author

ghost commented Dec 20, 2020

Like that?
image

@ghost
Copy link
Author

ghost commented Dec 29, 2020

able to merge?

@ericfennis
Copy link
Member

@delnyn Yess awesome work!

@ericfennis ericfennis merged commit cf13cef into lucide-icons:master Dec 30, 2020
@ghost ghost deleted the brush-icon branch December 30, 2020 20:27
@karsa-mistmere karsa-mistmere mentioned this pull request Jun 15, 2022
Open
8 tasks
This pull request was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
🎨 icon About new icons
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@ericfennis @locness3