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

How to deal with favicon? #650

Closed
deniak opened this issue Aug 8, 2024 · 3 comments · Fixed by w3c/w3c-website-templates-bundle#166
Closed

How to deal with favicon? #650

deniak opened this issue Aug 8, 2024 · 3 comments · Fixed by w3c/w3c-website-templates-bundle#166
Assignees

Comments

@deniak
Copy link
Member

deniak commented Aug 8, 2024

Describe the issue
Nowadays, browsers systematically try to fetch a favicon when loading pages. If no icon link tag is provided (see below), then the browser try to look for one at the root of the website which may result in a 404 if it doesn't exists (might be the case for some sub domains).

<link rel="shortcut icon" href="https://www.w3.org/favicon.ico">

The different templates in the design system don't include that icon link but it's probably good to add it even if the favicon exists at the root of w3.org. So even if the template is used outside www.w3.org (github.io, different subdomain, etc), we would still have the good W3C favicon.

@vivienlacourba
Copy link
Member

A concrete example is the authentication portal which uses the new W3C style but at a different domain https://auth.w3.org/

@simonrjones any specific recommendation for favicon or can we simply add the above line to the website template?

@simonrjones
Copy link
Member

hi @vivienlacourba yes I agree that is worth adding. The example HTML added by @deniak looks perfect to me.

If possible, can you add an icon for Apple, iOS uses a custom link rel type. Example:

<link rel="apple-touch-icon" sizes="180x180" href="/apple-touch-icon.png">

@vivienlacourba vivienlacourba assigned deniak and unassigned simonrjones Oct 18, 2024
@vivienlacourba
Copy link
Member

Assigning to @deniak for implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants