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 Laptop icon #88

Merged
merged 6 commits into from Oct 14, 2020
Merged

Add Laptop icon #88

merged 6 commits into from Oct 14, 2020

Conversation

ghost
Copy link

@ghost ghost commented Oct 8, 2020

I worked on the icon and there it is. (created a new branch in the fork)

@ghost ghost mentioned this pull request Oct 8, 2020
@johnletey
Copy link
Collaborator

@delnyn Maybe try this for the icon? 😄

<svg
  xmlns="http://www.w3.org/2000/svg"
  width="24"
  height="24"
  viewBox="0 0 24 24"
  fill="none"
  stroke="currentColor"
  stroke-width="2"
  stroke-linecap="round"
  stroke-linejoin="round"
>
  <rect x="3.81824" y="5" width="16.3636" height="10.5" />
  <path d="M4 15L2 19H22L20 15" />
</svg>

@ghost
Copy link
Author

ghost commented Oct 9, 2020

Oh yeah sorry I messed up lol

@johnletey
Copy link
Collaborator

@delnyn All good 👍🏻

@ghost
Copy link
Author

ghost commented Oct 9, 2020

Great!

@ericfennis
Copy link
Member

@delnyn I noticed that the icon is different than the icon we discussed in #77

image

@ghost
Copy link
Author

ghost commented Oct 13, 2020

What? "Frame" is what you have in the PR?

@ericfennis
Copy link
Member

The icon in this pr

@ghost
Copy link
Author

ghost commented Oct 13, 2020

Commited changes!

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

Nice, small changes!

icons/laptop.svg Outdated Show resolved Hide resolved
delnyn and others added 2 commits October 14, 2020 17:55
Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
Co-authored-by: Eric Fennis <eric.fennis@gmail.com>
@ghost
Copy link
Author

ghost commented Oct 14, 2020

✔️ Commited!

@ericfennis
Copy link
Member

Nice work @delnyn 🚀!

@ericfennis ericfennis merged commit b80c280 into lucide-icons:master Oct 14, 2020
@ghost ghost deleted the laptop-icon branch October 22, 2020 11:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants