{"action":"created","issue":{"url":"https://api.github.com/repos/whatwg/html/issues/1352","repository_url":"https://api.github.com/repos/whatwg/html","labels_url":"https://api.github.com/repos/whatwg/html/issues/1352/labels{/name}","comments_url":"https://api.github.com/repos/whatwg/html/issues/1352/comments","events_url":"https://api.github.com/repos/whatwg/html/issues/1352/events","html_url":"https://github.com/whatwg/html/issues/1352","id":157658318,"number":1352,"title":"Navigate improvements","user":{"login":"annevk","id":1544111,"avatar_url":"https://avatars.githubusercontent.com/u/1544111?v=3","gravatar_id":"","url":"https://api.github.com/users/annevk","html_url":"https://github.com/annevk","followers_url":"https://api.github.com/users/annevk/followers","following_url":"https://api.github.com/users/annevk/following{/other_user}","gists_url":"https://api.github.com/users/annevk/gists{/gist_id}","starred_url":"https://api.github.com/users/annevk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/annevk/subscriptions","organizations_url":"https://api.github.com/users/annevk/orgs","repos_url":"https://api.github.com/users/annevk/repos","events_url":"https://api.github.com/users/annevk/events{/privacy}","received_events_url":"https://api.github.com/users/annevk/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/whatwg/html/labels/topic:%20navigation","name":"topic: navigation","color":"009800"}],"state":"open","locked":false,"assignee":{"login":"annevk","id":1544111,"avatar_url":"https://avatars.githubusercontent.com/u/1544111?v=3","gravatar_id":"","url":"https://api.github.com/users/annevk","html_url":"https://github.com/annevk","followers_url":"https://api.github.com/users/annevk/followers","following_url":"https://api.github.com/users/annevk/following{/other_user}","gists_url":"https://api.github.com/users/annevk/gists{/gist_id}","starred_url":"https://api.github.com/users/annevk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/annevk/subscriptions","organizations_url":"https://api.github.com/users/annevk/orgs","repos_url":"https://api.github.com/users/annevk/repos","events_url":"https://api.github.com/users/annevk/events{/privacy}","received_events_url":"https://api.github.com/users/annevk/received_events","type":"User","site_admin":false},"milestone":null,"comments":4,"created_at":"2016-05-31T12:22:11Z","updated_at":"2016-06-21T13:59:30Z","closed_at":null,"body":"Improvements I'd like to make to the navigate algorithm:\r\n\r\n- [ ] Redo the \"resource\" argument. Introduce a \"destination record\" concept (a URL record, session history entry, request, or response) and make it clear that the \"destination\" argument is one of those. (The word \"resource\" trips up the server folks and there's no reason to use it here, especially with such a narrow definition.) Note that the only case I found for supporting session history entries has been the \"entry update\" concept which this could replace. It's conceivable we could maybe update an entry without doing the whole navigate dance. Not sure I care personally in which order that is figured out.\r\n- [ ] Put \"Initialising a new Document object\" in its own distinct algorithm separate from navigate and make it more clear what arguments need to get passed to it. Probably give it a section of its own.\r\n- [ ] We need to have a _response_ variable set to null early on (before \"Fragments:\") that can be updated by various concepts along the way.\r\n- [ ] \"Handle redirects:\" should only jump to \"Fragments:\" for non-HTTP(S) scheme URLs and should at least set response to a network error for an attempt to redirect to a javascript URL. For HTTP(S) scheme URLs the HTTP-redirect fetch defined in Fetch needs to be used. Parsing the `Location` header needs to be abstracted and coordinated with Fetch (I might favor a special response that has an additional field to store the parsed result). Other than data URLs it seems Firefox supports no other schemes here either, but my testing has not been exhaustive (mailto, facetime, about). (#461)\r\n- [ ] The navigate algorithm steps from the step that starts with \"Resource handling:\" onward should be its own algorithm so that multipart/x-mixed-replace (though is that still supported?) can invoke it directly. Should probably first check multipart/x-mixed-replace is still a thing. Now #1353.\r\n- [ ] If we don't merge the two places that initialize a navigated document, make sure to rename \"implement the sandboxing\" per https://github.com/whatwg/html/issues/1385#issuecomment-226334073."},"comment":{"url":"https://api.github.com/repos/whatwg/html/issues/comments/227448331","html_url":"https://github.com/whatwg/html/issues/1352#issuecomment-227448331","issue_url":"https://api.github.com/repos/whatwg/html/issues/1352","id":227448331,"user":{"login":"annevk","id":1544111,"avatar_url":"https://avatars.githubusercontent.com/u/1544111?v=3","gravatar_id":"","url":"https://api.github.com/users/annevk","html_url":"https://github.com/annevk","followers_url":"https://api.github.com/users/annevk/followers","following_url":"https://api.github.com/users/annevk/following{/other_user}","gists_url":"https://api.github.com/users/annevk/gists{/gist_id}","starred_url":"https://api.github.com/users/annevk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/annevk/subscriptions","organizations_url":"https://api.github.com/users/annevk/orgs","repos_url":"https://api.github.com/users/annevk/repos","events_url":"https://api.github.com/users/annevk/events{/privacy}","received_events_url":"https://api.github.com/users/annevk/received_events","type":"User","site_admin":false},"created_at":"2016-06-21T13:59:30Z","updated_at":"2016-06-21T13:59:30Z","body":"The initialize setup above is broken. We cannot create a document before we have a global to create one in.\r\n\r\nI suspect what we need to do is pull out creating an global. Move that up and share it with create a browsing context.\r\n\r\nThen have something to create a document. Again, try to make that an algorithm that both navigate and create a browsing context can use.\r\n\r\nThen, hook up the document to the global. Can maybe be combined with previous step.\r\n\r\nThen, for the navigate case, dispatch to the various algorithms that tell how to parse the HTML / handle the image / display the error / etc. and let those fill in the remaining document details."}}
{"actor":{"display_login":"annevk"}}