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 logo and favicon #563

Merged
merged 1 commit into from
Jul 26, 2017
Merged

Add logo and favicon #563

merged 1 commit into from
Jul 26, 2017

Conversation

defect
Copy link
Contributor

@defect defect commented Jul 17, 2017

This adds the logo @michaeljs1990 kindly gave collins in #542. Thank you! The favicon file contains 16x16, 32x32 and 64x64 sizes. I wasn't really sure where to put the original image so i just added it next to the favicon.

I had to add a little bit of horizontal space for the image to be square. Here's the full process i used in case anyone wants to tweak the paramaters:

convert logo.png -gravity East -background transparent -splice 175x0 step1.png
convert step1.png -gravity West -background transparent -splice 72x0 step2.png

for geom in "16x16" "32x32" "64x64"; do
    convert step2.png -thumbnail "$geom" "$geom.png"
    optipng "$geom.png"
done

convert 16x16.png 32x32.png 64x64.png favicon.ico
rm 16x16.png 32x32.png 64x64.png step1.png step2.png

In the end, it looks like this in chrome:
Screenshot

@michaeljs1990 @byxorna @roymarantz

Copy link
Contributor

@byxorna byxorna left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like hot 🔥 !

@michaeljs1990
Copy link
Contributor

niceeee, thanks for putting up the PR!

@michaeljs1990
Copy link
Contributor

Fun fact, the logo cost me 0.5 Bitcoin but at the time of purchase that was only ~100$.

@defect defect merged commit a849584 into tumblr:master Jul 26, 2017
@defect defect mentioned this pull request Jul 26, 2017
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.

4 participants