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: add <div algorithm> wrappers to Header class #1596
Conversation
fetch.bs
Outdated
<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method steps are: | ||
<div algorithm> | ||
<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method | ||
from a {{Headers}} object (<var>headers</var>), run these steps: |
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.
You found a bug, nice! The fix is replacing headers below with <a>this</a>
. This can stay as-is. I suppose the same applies to set()
below.
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.
Ah! Good point. Originally I thought this would be like remove privileged no-CORS request-headers, but it makes sense that we'd use this
for these methods.
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.
Thanks for the review! I'll try to update this evening.
fetch.bs
Outdated
<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method steps are: | ||
<div algorithm> | ||
<p>The <dfn export for=Headers method><code>delete(<var>name</var>)</code></dfn> method | ||
from a {{Headers}} object (<var>headers</var>), run these steps: |
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.
Ah! Good point. Originally I thought this would be like remove privileged no-CORS request-headers, but it makes sense that we'd use this
for these methods.
Updated to use |
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.
\o/
Related to: #1526
Preview | Diff