Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@berniegp
Copy link

@berniegp berniegp commented Aug 26, 2022

The spec uses the term "author request headers" everywhere except in the description of the OPENED state in https://xhr.spec.whatwg.org/#states. This commit fixes this inconsistency.

I think this qualifies as an Editorial change, but I am unsure. Please advise is this is incorrect.

  • At least two implementers are interested (and none opposed):
  • Tests are written and can be reviewed and commented upon at:
  • Implementation bugs are filed:
    • Chrome: …
    • Firefox: …
    • Safari: …

(See WHATWG Working Mode: Changes for more details.)


Preview | Diff

@annevk
Copy link
Member

annevk commented Aug 29, 2022

I think this is actually fine and instead we should reword

Combines a header in author request headers.

to something like:

Appends a value to an existing request header or adds a new request header.

"author request headers" is an internal field of XMLHttpRequest instances and there's no need to talk about it in informative text.

@berniegp
Copy link
Author

I wasn't aware of the possible distinct treatment of informative text. In that case, I agree that your suggestion is much clearer than talking about "author" request headers.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants