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.

Sign up for GitHub

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

Jump to bottom

initiatorType for favicon #298

Open
noamr opened this issue Oct 3, 2021 · 6 comments
Open

initiatorType for favicon #298

noamr opened this issue Oct 3, 2021 · 6 comments
Projects
Triage

Comments

@noamr
Copy link
Contributor

noamr commented Oct 3, 2021 •
edited
Loading

Currently it's "other" for chrome and Firefox, and "icon" for Safari.
According the spec, it's vague what it should be (maybe "link" since it's originated from a link tag?)
It's difficult to write a WPT for it since it's not straightforward to force favicon loading in Safari.
@yoavweiss
Copy link
Contributor

Agree it'd be good to align here, but unclear on what value.
/cc @npm1 @achristensen07 @bdekoz
@npm1
Copy link
Contributor

npm1 commented Oct 4, 2021

Other than Safari being hard, did you find what values are returned on Chrome/Firefox?
@noamr
Copy link
Contributor Author

noamr commented Oct 4, 2021

Other than Safari being hard, did you find what values are returned on Chrome/Firefox?

It returns "other" in Chrome and Firefox.
@achristensen07
Copy link
Contributor

I don't think it would be a big deal to change Safari to say "other" here
@noamr
Copy link
Contributor Author

noamr commented Oct 5, 2021

See also #299. I wonder if favicon should have a resource timing entry at all, given that it's not accessible in the document in any other way.
@yoavweiss
Copy link
Contributor

This was discussed on the WG call.
@yoavweiss yoavweiss added this to Triage in Triage May 5, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
4 participants
@noamr @yoavweiss @achristensen07 @npm1