-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Closed
Labels
area-mvcIncludes: MVC, Actions and Controllers, Localization, CORS, most templatesIncludes: MVC, Actions and Controllers, Localization, CORS, most templatesbugThis issue describes a behavior which is not expected - a bug.This issue describes a behavior which is not expected - a bug.good first issueGood for newcomers.Good for newcomers.help wantedUp for grabs. We would accept a PR to help resolve this issueUp for grabs. We would accept a PR to help resolve this issue
Milestone
Description
The favicon.ico file currently used in all of our project templates is pretty big by favicon standards at 31.2 KB. In fact, yslow recommends a size less than 1 KB, however a quick look at some big properties shows sizes bigger than that but still much smaller than our default:
Site | Icon size |
---|---|
google.com | 5.3 KB |
microsoft.com | 17.2 KB |
stackoverflow.com | 5.5 KB |
our template | 31.2 KB |
Cracking open the Google favicon.ico reveals they have only two sizes embedded: 32x32@32-bit and 16x16@32-bit. Our default has two more sizes beyond that: 48x48 and 64x64.
Reducing ours to match Google's (seems like a reasonable default) drops the size to match theirs (as expected) at 5.3 KB so we should likely just do that.
The smaller, optimized ICO file is in this ZIP: favicon.zip
martincostello, slang25, st0le, NickCraver, henriksen and 14 more
Metadata
Metadata
Assignees
Labels
area-mvcIncludes: MVC, Actions and Controllers, Localization, CORS, most templatesIncludes: MVC, Actions and Controllers, Localization, CORS, most templatesbugThis issue describes a behavior which is not expected - a bug.This issue describes a behavior which is not expected - a bug.good first issueGood for newcomers.Good for newcomers.help wantedUp for grabs. We would accept a PR to help resolve this issueUp for grabs. We would accept a PR to help resolve this issue