Skip to content

Conversation

@ahtohbi4
Copy link
Contributor

@ahtohbi4 ahtohbi4 commented Dec 2, 2020

Icon "timer" was added.

Icon timer

@ericfennis
Copy link
Member

Nice, love it! 🔥

ericfennis
ericfennis previously approved these changes Dec 2, 2020
Co-authored-by: Eric Fennis <[email protected]>
@moeenio
Copy link
Contributor

moeenio commented Dec 3, 2020

I think the middle part could be bigger, even if that requires not respecting the 2px safe zone rule

@ericfennis
Copy link
Member

@locness3 Well spotted.

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.

@locness3 with 2px spacing 🤔
image
image

@moeenio
Copy link
Contributor

moeenio commented Dec 9, 2020

Right looks good.

@ericfennis ericfennis added the 🎨 icon About new icons label Dec 20, 2020
@ericfennis ericfennis dismissed their stale review December 20, 2020 15:19

please add the 2px spacing

@ericfennis ericfennis merged commit ce80469 into lucide-icons:master Dec 22, 2020
@karsa-mistmere karsa-mistmere mentioned this pull request Jun 15, 2022
10 tasks
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.

3 participants