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
Copy Markdown
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
Copy Markdown
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!

Comment thread source Outdated
Comment thread source Outdated
Comment thread source Outdated
Comment thread source Outdated
Comment thread source
@domfarolino domfarolino requested a review from annevk January 7, 2021 23:08
@domfarolino
Copy link
Copy Markdown
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