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

New application icon #390

Merged
merged 3 commits into from
Mar 13, 2019
Merged

New application icon #390

merged 3 commits into from
Mar 13, 2019

Conversation

phw
Copy link
Owner

@phw phw commented Nov 4, 2018

See #114

@phw phw mentioned this pull request Nov 4, 2018
Closed
@gort818
Copy link
Contributor

gort818 commented Nov 7, 2018

Looks awesome! Thanks @bertob

@phw
Copy link
Owner Author

phw commented Dec 29, 2018

@snwh, @jimmac, @lapo: How would you like to be credited here for the icon? Thanks for your work, it's awesome!

@jimmac
Copy link

jimmac commented Jan 9, 2019

Is this blocking on vanity? ;)

The aboutbox is quite exposed and perfect fine from my POV.

@snwh
Copy link

snwh commented Jan 9, 2019

About box/name only credit is fine by me as well.

@bertob
Copy link

bertob commented Jan 28, 2019

Can we merge this? :)

@phw phw merged commit 1ae0e0c into master Mar 13, 2019
@phw phw deleted the 114-new-icon branch March 13, 2019 10:41
dz0ny pushed a commit to dz0ny/peek that referenced this pull request Oct 2, 2019
@phw @dz0ny
New application icon (phw#390)
5e9efbe
dz0ny pushed a commit to dz0ny/peek that referenced this pull request Dec 1, 2020
@phw @dz0ny
New application icon (phw#390)
4e31f95
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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@phw @gort818 @jimmac @snwh @bertob