Skip to content

Editorial: Introduce #concept-document-origin#6249

Merged
annevk merged 7 commits intomasterfrom
domfarolino/introduce-concept-document-origin
Jan 12, 2021
Merged

Editorial: Introduce #concept-document-origin#6249
annevk merged 7 commits intomasterfrom
domfarolino/introduce-concept-document-origin

Conversation

@domfarolino
Copy link
Member

@domfarolino domfarolino commented Dec 25, 2020

Fixes #1725.

So far there are three commits to this PR:

  1. Adding a dummy dfn for #concept-document-origin to link to later
  2. Linking to it from every instance of "origin" in the HTML Standard that seemed to specifically reference a Document's origin (it is possible I missed a couple, but pretty sure I got most or all of them)
  3. Finish up the paragraph for the dfn of #concept-document-origin, explaining when an origin is changed, and how this origin can differ from the origin of a Document's URL
    • Also making a few more instances of "origin" point to the new dfn specifically

I know the origin is changed at least in document.domain, which I mention in my dfn paragraph. It's possible there are more (and I'd like to know about them), but I don't think we need to make the "for example ..." exhaustive.

As a bonus, I think I got all of the wrapping right (but would like someone to take a look)... just in time for Christmas eh? (ba dum tsss)


/browsers.html ( diff )
/browsing-the-web.html ( diff )
/comms.html ( diff )
/dom.html ( diff )
/dynamic-markup-insertion.html ( diff )
/history.html ( diff )
/iframe-embed-object.html ( diff )
/images.html ( diff )
/interaction.html ( diff )
/interactive-elements.html ( diff )
/links.html ( diff )
/origin.html ( diff )
/parsing.html ( diff )
/web-messaging.html ( diff )
/webstorage.html ( diff )
/window-object.html ( diff )

Copy link
Member

@annevk annevk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks so much for tackling this!

@domfarolino domfarolino requested a review from annevk January 7, 2021 23:08
@domfarolino
Copy link
Member Author

domfarolino commented Jan 8, 2021

OK I think everything here has been addressed (assuming I'm understanding the "resolve" on this thread), feel free to take another look!

@domfarolino domfarolino changed the title Introduce #concept-document-origin Editorial: Introduce #concept-document-origin Jan 12, 2021
@domfarolino domfarolino added clarification Standard could be clearer topic: origin labels Jan 12, 2021
@annevk annevk merged commit 6881195 into master Jan 12, 2021
@annevk annevk deleted the domfarolino/introduce-concept-document-origin branch January 12, 2021 07:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

clarification Standard could be clearer topic: origin

Development

Successfully merging this pull request may close these issues.

Reference document's origin accurately

3 participants