RealFaviconGenerator / realfavicongenerator

Generate favicon for all major platforms
501 stars 24 forks source link

Why does RFG generate a 48x48 version inside the .ico file? #155

Open tigt opened 9 years ago

tigt commented 9 years ago

From the FAQ:

The generated favicon.ico contains the icon in 16x16, 32x32 and 48x48. You may wonder what happens in extreme cases, like a 300% zoom in Windows 8.1 in HiDPI. It appears that 32x32 is enough.

And later:

He also tried high resolution on MacOS with Chrome, Opera 15+, Opera 12.16, Safari, OmniWeb and iCab: none of them used a resolution higher than 32x32.

If the 48x48 version isn't used by anything, why is it included?

phbernard commented 9 years ago

You're right, explanations are not clear. In addition to the FAQ, I would like to mention that the compatibility test results reveal no usage for this icon. So why including it?

Because Microsoft recommends it. As I only have Windows 8 to do testing, I can't rely on my observations alone to pretend the 48x48 icon is useless. So I keep it. This is an opinionated approach.

That may change with Windows 10: since this version will be mostly free, it should be widely adopted. And if this version does not use the 48x48 icon at all, maybe I'll remove it. So let's talk about this again in a year :)

tigt commented 9 years ago

If you like, I could test on Windows 7; I doubt it's much different from Vista, and XP probably isn't supported. On Apr 22, 2015 4:25 AM, "Philippe Bernard" notifications@github.com wrote:

You're right, explanations are not clear. In addition to the FAQ, I would like to mention that the compatibility test results http://realfavicongenerator.net/favicon_compatibility reveal no usage for this icon. So why including it?

Because Microsoft recommends it. As I only have Windows 8 to do testing, I can't rely on my observations alone to pretend the 48x48 icon is useless. So I keep it. This is an opinionated approach.

That may change with Windows 10: since this version will be mostly free, it should be widely adopted. And if this version does not use the 48x48 icon at all, maybe I'll remove it. So let's talk about this again in a year :)

— Reply to this email directly or view it on GitHub https://github.com/RealFaviconGenerator/realfavicongenerator/issues/155#issuecomment-95072039 .

phbernard commented 9 years ago

Yep, I'm interested! It's all here: http://realfavicongenerator.net/favicon_compatibility_test

tigt commented 9 years ago

Found it! Steps to reproduce:

However, the 48x48 doesn't seem to fit in the space well at all; it's 96x96! It adds a bunch of margin around it to compensate.

When used Medium Icons in Desktop View menu, the available space is 48x48, but it uses the 32x32 icon on a graphic of a folded page instead.

This is weirder than I could have guessed On Apr 22, 2015 1:36 PM, "Philippe Bernard" notifications@github.com wrote:

Yep, I'm interested! It's all here: http://realfavicongenerator.net/favicon_compatibility_test

— Reply to this email directly or view it on GitHub https://github.com/RealFaviconGenerator/realfavicongenerator/issues/155#issuecomment-95278875 .

tigt commented 9 years ago

Wait, even worse; it uses the 16x16 with the page graphic, not the 32x32

tigt commented 9 years ago

Messing with it further, it looks like there's problems with caching affecting the outcome, things are even harder to track down