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

feat: Add timer icon #148

Merged
merged 3 commits into from
Dec 22, 2020
Merged

feat: Add timer icon #148

merged 3 commits into from
Dec 22, 2020

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
tags.json Outdated Show resolved Hide resolved
Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
@locness3
Copy link
Contributor

locness3 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

@locness3
Copy link
Contributor

locness3 commented Dec 9, 2020

Right looks good.

icons/timer.svg Outdated Show resolved Hide resolved
@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
8 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.

None yet

3 participants