{"action":"created","issue":{"url":"https://api.github.com/repos/drufball/directory-download/issues/2","repository_url":"https://api.github.com/repos/drufball/directory-download","labels_url":"https://api.github.com/repos/drufball/directory-download/issues/2/labels{/name}","comments_url":"https://api.github.com/repos/drufball/directory-download/issues/2/comments","events_url":"https://api.github.com/repos/drufball/directory-download/issues/2/events","html_url":"https://github.com/drufball/directory-download/issues/2","id":162856235,"number":2,"title":"Does not make sense to accept *both* File-list and DirectoryEntry","user":{"login":"sicking","id":1831914,"avatar_url":"https://avatars.githubusercontent.com/u/1831914?v=3","gravatar_id":"","url":"https://api.github.com/users/sicking","html_url":"https://github.com/sicking","followers_url":"https://api.github.com/users/sicking/followers","following_url":"https://api.github.com/users/sicking/following{/other_user}","gists_url":"https://api.github.com/users/sicking/gists{/gist_id}","starred_url":"https://api.github.com/users/sicking/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/sicking/subscriptions","organizations_url":"https://api.github.com/users/sicking/orgs","repos_url":"https://api.github.com/users/sicking/repos","events_url":"https://api.github.com/users/sicking/events{/privacy}","received_events_url":"https://api.github.com/users/sicking/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2016-06-29T07:13:28Z","updated_at":"2016-07-27T02:24:28Z","closed_at":null,"body":"The explainer says\r\n\r\n> Navigator.saveDirectory() could accept a list of File objects, a DirectoryEntry, or both.\r\n\r\nIt does not make sense to accept both a list of File objects and a DirectroyEntry at the same time. It's not even clear to me what syntax the \"both\" option intends to use. It is more correct to say \"Navigator.saveDirectory() could accept a list of File objects, or a DirectoryEntry.\"\r\n\r\nAlternatively, we could allow the list to contain both File objects and DirectoryEntry objects. In WebIDL terms, I would say that saveDirectory would use\r\n\r\n```\r\npartial interface Navigator {\r\n  void saveDirectory((sequence<(File or DirectoryEntry)> or DirectoryEntry), optional DOMString nameHint);\r\n};\r\n```\r\n\r\nIf a sequence is *not* used, and the nameHint argument is left out, it defaults to the name of the DirectoryEntry."},"comment":{"url":"https://api.github.com/repos/drufball/directory-download/issues/comments/235463389","html_url":"https://github.com/drufball/directory-download/issues/2#issuecomment-235463389","issue_url":"https://api.github.com/repos/drufball/directory-download/issues/2","id":235463389,"user":{"login":"drufball","id":5472050,"avatar_url":"https://avatars.githubusercontent.com/u/5472050?v=3","gravatar_id":"","url":"https://api.github.com/users/drufball","html_url":"https://github.com/drufball","followers_url":"https://api.github.com/users/drufball/followers","following_url":"https://api.github.com/users/drufball/following{/other_user}","gists_url":"https://api.github.com/users/drufball/gists{/gist_id}","starred_url":"https://api.github.com/users/drufball/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/drufball/subscriptions","organizations_url":"https://api.github.com/users/drufball/orgs","repos_url":"https://api.github.com/users/drufball/repos","events_url":"https://api.github.com/users/drufball/events{/privacy}","received_events_url":"https://api.github.com/users/drufball/received_events","type":"User","site_admin":false},"created_at":"2016-07-27T02:24:28Z","updated_at":"2016-07-27T02:24:28Z","body":"@sicking Good catch! I didn't realize I'd said they could be passed together. Do you think there's a specific ergonomics reason to expect both simultaneously? Otherwise it might be simpler to accept just one at a time."}}
{"actor":{"display_login":"drufball"}}