- From: Domenic Denicola <notifications@github.com>
- Date: Thu, 08 May 2025 23:15:30 -0700
- To: whatwg/fetch <fetch@noreply.github.com>
- Cc: Subscribed <subscribed@noreply.github.com>
- Message-ID: <whatwg/fetch/pull/1823@github.com>
After much discussion in #1820 and #1821, this seems to be more correct. Closes #1820. Closes #1821. Helps https://github.com/whatwg/html/pull/11250 specify browser-initiated navigations better. <!-- Thank you for contributing to the Fetch Standard! Please describe the change you are making and complete the checklist below if your change is not editorial. When you submit this PR, and each time you edit this comment (including checking a checkbox through the UI!), PR Preview will run and update it. As such make any edits in one go and only after PR Preview has run. If you think your PR is ready to land, please double-check that the build is passing and the checklist is complete before pinging. --> - [x] At least two implementers are interested (and none opposed): * This seems to match all browser behavior. - [ ] [Tests](https://github.com/web-platform-tests/wpt) are written and can be reviewed and commented upon at: * Not really testable in WPT, due to https://github.com/web-platform-tests/wpt/issues/16019 I believe. - [x] [Implementation bugs](https://github.com/whatwg/meta/blob/main/MAINTAINERS.md#handling-pull-requests) are filed: * Chromium: N/A * Gecko: N/A, although maybe a bug to encourage the blanking-out behavior would be appreciated? * WebKit: N/A, although maybe a bug to encourage the blanking-out behavior would be appreciated? * Deno (not for CORS changes): N/A - [x] [MDN issue](https://github.com/whatwg/meta/blob/main/MAINTAINERS.md#handling-pull-requests) is filed: N/A - [x] The top of this comment includes a [clear commit message](https://github.com/whatwg/meta/blob/main/COMMITTING.md) to use. <!-- If you created this PR from a single commit, Github copied its message. Otherwise, you need to add a commit message yourself. --> (See [WHATWG Working Mode: Changes](https://whatwg.org/working-mode#changes) for more details.) --- Marking as "do not merge yet" as we should prepare PRs for all of https://dontcallmedom.github.io/webdex/w.html#window%40%40request%40dfn . But I don't want to do that until we've confirmed this is the right path, since it's my third attempt :sweat_smile: <!-- This comment and the below content is programmatically generated. You may add a comma-separated list of anchors you'd like a direct link to below (e.g. #idl-serializers, #idl-sequence): Don't remove this comment or modify anything below this line. If you don't want a preview generated for this pull request, just replace the whole of this comment's content by "no preview" and remove what's below. --> *** <a href="https://whatpr.org/fetch/1823.html" title="Last updated on May 9, 2025, 6:15 AM UTC (cd1da21)">Preview</a> | <a href="https://whatpr.org/fetch/1823/5a96806...cd1da21.html" title="Last updated on May 9, 2025, 6:15 AM UTC (cd1da21)">Diff</a> You can view, comment on, or merge this pull request online at: https://github.com/whatwg/fetch/pull/1823 -- Commit Summary -- * Replace request's "window" with "traversable for user prompts" -- File Changes -- M fetch.bs (131) -- Patch Links -- https://github.com/whatwg/fetch/pull/1823.patch https://github.com/whatwg/fetch/pull/1823.diff -- Reply to this email directly or view it on GitHub: https://github.com/whatwg/fetch/pull/1823 You are receiving this because you are subscribed to this thread. Message ID: <whatwg/fetch/pull/1823@github.com>
Received on Friday, 9 May 2025 06:15:34 UTC