-
Notifications
You must be signed in to change notification settings - Fork 323
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
Editorial: align with URL's cannot-be-base-URL removal #1337
Conversation
These URLs now have a path whose value is a string rather than a list. See whatwg/url#655 for context.
Tests: web-platform-tests/wpt#31305. Fetch PR: whatwg/fetch#1337. HTML PR: whatwg/html#7240. URLPattern PR: whatwg/urlpattern#143. Follow-up: #659. Fixes #634.
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 with optional suggestion that I'm not too sure about myself.
fetch.bs
Outdated
<a for=response>header list</a> consist of a single <a for=/>header</a> whose | ||
<a for=header>name</a> is `<code>Content-Type</code>` and <a for=header>value</a> is | ||
`<code>text/html;charset=utf-8</code>`, and <a for=response>body</a> is the empty byte sequence. | ||
<p>If <var>request</var>'s <a for=request>current URL</a>'s <a for=url>path</a> is |
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.
Maybe the string "blank"
for extra clarity? It's redundant but might help a bit.
fetch.bs
Outdated
is "<code>about</code>", and <a for=url>path</a> contains a single string | ||
"<code>client</code>" | ||
<li><p><var>parsedReferrer</var>'s <a for=url>scheme</a> is "<code>about</code>" and | ||
<a for=url>path</a> is "<code>client</code>" |
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.
Similarly here.
I'm fine with trying that for a bit. Normally I end up removing those type restatements, but these URLs are not too frequent so maybe it can help. |
These URLs now have a path whose value is a string rather than a list.
See whatwg/url#655 for context.
Preview | Diff