Skip to content

Add brush icon #108

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
Merged
ericfennis merged 9 commits into from Dec 30, 2020
Merged

Add brush icon #108

ericfennis 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

ericfennis commented Oct 25, 2020

@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

ericfennis commented Oct 31, 2020

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

ericfennis commented Nov 14, 2020

@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

ericfennis commented Nov 17, 2020

@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

ericfennis commented Dec 20, 2020

@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

ericfennis commented Dec 30, 2020

@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
10 tasks
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 @moeenio