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
Fixes #1337 - Align on HTML terminology for origins #1416
Conversation
draft-ietf-httpbis-rfc6265bis.md
Outdated
context" are defined in {{HTML}}. | ||
"dedicated worker", "Document", "nested browsing context", "opaque origin", | ||
"parent browsing context", "sandboxed origin browsing context flag", "shared | ||
worker", "the worker's Documents", and "top-level browsing context", |
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.
gross, added in an extra space unintentionally 😞
54bd550
to
5921b27
Compare
draft-ietf-httpbis-rfc6265bis.md
Outdated
defined in {{HTML}}. | ||
"dedicated worker", "Document", "nested browsing context", "opaque origin", | ||
"parent browsing context", "sandboxed origin browsing context flag", "shared | ||
worker", "the worker's Documents", and "top-level browsing context", |
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.
Super minor nit, the "and" should go right before the penultimate list element.
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.
oh, alphabetize fail. thanks.
Lgtm % nit |
5921b27
to
305a9c0
Compare
Thanks @chlily1. I fixed the "English bug", feel free to merge when ready. |
305a9c0
to
a1dc719
Compare
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.
LGTM.
It looks like Steven got rid of "scheme/host/port triple" in #1348, so this just covers using "opaque origin".
PTAL @chlily1 @englehardt
(changelog commit will get pushed as soon as I have a PR URI)