Skip to content

[import-bytes] add bytes sec-fetch-dest and accept#1912

Open
styfle wants to merge 2 commits intowhatwg:mainfrom
styfle:styfle/import-bytes
Open

[import-bytes] add bytes sec-fetch-dest and accept#1912
styfle wants to merge 2 commits intowhatwg:mainfrom
styfle:styfle/import-bytes

Conversation

@styfle
Copy link
Copy Markdown
Contributor

@styfle styfle commented Mar 5, 2026

@styfle styfle marked this pull request as ready for review March 5, 2026 02:28
@styfle styfle changed the title [import-bytes] add "bytes" sec-fetch-dest and accept [import-bytes] add bytes sec-fetch-dest and accept Mar 5, 2026
fetch.bs Outdated
"<code>bytes</code>",
"<code>font</code>", "<code>image</code>", "<code>json</code>", "<code>manifest</code>",
"<code>paintworklet</code>", "<code>script</code>", "<code>style</code>", "<code>track</code>",
"<code>video</code>", "<code>xslt</code>", or the empty string.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

We should rewrap here.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Sounds good, I adjusted the line wrap 👍

fetch.bs Outdated
<a>implementation-defined</a> number of times, <a>in parallel</a> from each other, and wait for
at least 1 to return a value. In an <a>implementation-defined</a> manner, select a value to
return from the returned values and return it. Any other returned values that are
<a>connections</a> may be closed.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Indentation got lost here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Good catch, fixed 👍

fetch.bs Outdated
<dd>`<code>application/json,*/*;q=0.5</code>`

<dt>"<code>bytes</code>"
<dd>`<code>*/*</code>`
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Let's put this first so they are in lexicographical order.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Ah nice I missed that. Fixed 👍

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