-
Notifications
You must be signed in to change notification settings - Fork 4
Open
Labels
enhancementNew feature or requestNew feature or request
Description
The WEKit already does some things right and recommends only a few selected favicons - compared to the mass provided by favicon generators.
This article is a good guideline: https://evilmartians.com/chronicles/how-to-favicon-in-2021-six-files-that-fit-most-needs#the-extremely-short-version
- Our manifest file already follows this recommendation and only references two icon sizes ✅
{
"icons": [
{ "src": "/icon-192.png", "type": "image/png", "sizes": "192x192" },
{ "src": "/icon-512.png", "type": "image/png", "sizes": "512x512" }
]
}
-
We can delete the
favicon-32x32.pngfile and remove the reference in head.html. -
Link
favicon.icoin head.html. Browsers actually automatically look in the root for the file, but we can then add asizes="any"attribute. This fixes a bug on Chrome that otherwise prefers ICO instead of SVG. -
Our
favicon.icois 16x16 in size. Recommended is 32x32. -
TBD: Document this recommendation or link the article. Otherwise, the favicon generators are quickly used again.
Metadata
Metadata
Assignees
Labels
enhancementNew feature or requestNew feature or request