-
Notifications
You must be signed in to change notification settings - Fork 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
Referrer-Policy - Update for "Safari on iOS", Safari and IE (#14675) #15089
Conversation
@ddbeck If you don't mind, I'd appreciate it if you could take a look. It would be great if you could merge it or comment. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi and thank you for this PR. I see it's your first to the project—welcome!
I think the IE details may be misplaced. See the line comment for more detail
Thank you for reviewing. I overlooked that https://developer.mozilla.org/ja/docs/Web/HTML/Element/meta was there. There seems to be no information about referrers other than IE, so I'll investigate that while I'm at it. |
I have checked on the corresponding versions of each.
And I also referred to https://caniuse.com/referrer-policy I could not test Opera Android because I do not have the environment to test it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good. Thank you! 🎉
Summary
Update for "Safari on iOS", Safari and IE.
The "notes" in IE is reprinted from https://caniuse.com/referrer-policy
Test results and supporting details
see #14675
Related issues
Fixes #14675