Skip to content

Comments

Add icon#52

Merged
MorrisJobke merged 2 commits intomasterfrom
proper-icon
Jan 20, 2017
Merged

Add icon#52
MorrisJobke merged 2 commits intomasterfrom
proper-icon

Conversation

@MorrisJobke
Copy link
Member

@MorrisJobke MorrisJobke commented Jan 19, 2017

* was done by Felix (https://github.com/eppfel)
* I inverted it and ran an SVG optimizer
* see #33 (comment)
* fixed #33

Signed-off-by: Morris Jobke <[email protected]>
@MorrisJobke
Copy link
Member Author

cc @skjnldsv @jancborchardt

@skjnldsv
Copy link
Member

I like it. Seemed a little bit out of place, but I think this is because the icons are too big on the app store. Doesn't matter :)

@eppfel
Copy link
Member

eppfel commented Jan 19, 2017

I found out what was wrong and fixed it: 😁
bildschirmfoto 2017-01-20 um 00 44 35

Signed-off-by: Felix A. Epp <[email protected]>
@skjnldsv
Copy link
Member

MUCH BETTER! 🎉

@MorrisJobke MorrisJobke merged commit e078d21 into master Jan 20, 2017
@jancborchardt jancborchardt deleted the proper-icon branch January 24, 2017 12:29
@jancborchardt
Copy link
Member

Good modification @eppfel! :)

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.

Needs a proper icon

4 participants