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 higher resolution icons to favicon.ico #1121

Merged
merged 1 commit into from Mar 13, 2014

Conversation

omega
Copy link
Contributor

@omega omega commented Mar 12, 2014

Adds a 32x32 and 48x48 icon to favicon.ico. It means the filesize increases slightly,
but it also means that it looks nicer in Firefox, Safari and Chrome tabs.

I also added apple-touch-icon (152x152, iOS7 retina version) if someone saves metacpan
to their "home screen" on mobile devices, this icon will be used.

The favicon.ico icons are transparent, the apple-touch-icon is on a white background.
iOS uses a black background for transparent parts and that looked pretty damn awful.

New firefox tab look

Old firefox tab look

Adds a 32x32 and 48x48 icon to favicon.ico. It means the filesize increases slightly,
but it also means that it looks nicer in Firefox, Safari and Chrome tabs.

I also added apple-touch-icon (152x152, iOS7 retina version) if someone saves metacpan
to their "home screen" on mobile devices, this icon will be used.

The favicon.ico icons are transparent, the apple-touch-icon is on a white background.
iOS uses a black background for transparent parts and that looked pretty damn awful.
@oalders
Copy link
Member

oalders commented Mar 13, 2014

Thanks very much! 👍

oalders added a commit that referenced this pull request Mar 13, 2014
Add higher resolution icons to favicon.ico
@oalders oalders merged commit 48df1cd into metacpan:master Mar 13, 2014
@omega omega deleted the hires-favicon branch March 14, 2014 04:07
oiami pushed a commit to oiami/metacpan-web that referenced this pull request Jun 9, 2014
Add higher resolution icons to favicon.ico
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