-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Wrong favicon type #393
Comments
Hi! Tho, we can move havicon template into own partial, not a big deal |
You can use this https://realfavicongenerator.net/favicon_checker to feel the complexity and misfortune of the fate of modern website favicon. 😔 I start thinking about auto-generating the missed favicon sizes by Hugo in the some sort of partial template. Recommended sizes are 16x16, and 32x32 for legacy Windows, 120x120 for Android, 180x180 for Apple etc... |
In 69886cc favicon was moved to separate shortcode to be able to customize, should be good enough |
In the file:
layouts\partials\docs\html-head.html
line 13:has wrong
type
attribute, it must be eitherimage/png
or file type must be.ico
correspondingly.P.S. Also in the other themes I noticed a favicon stuff placed in the separate
layouts\partials\favicon.html
file.The text was updated successfully, but these errors were encountered: