{"action":"opened","issue":{"url":"https://api.github.com/repos/d3/d3-selection/issues/77","repository_url":"https://api.github.com/repos/d3/d3-selection","labels_url":"https://api.github.com/repos/d3/d3-selection/issues/77/labels{/name}","comments_url":"https://api.github.com/repos/d3/d3-selection/issues/77/comments","events_url":"https://api.github.com/repos/d3/d3-selection/issues/77/events","html_url":"https://github.com/d3/d3-selection/issues/77","id":159180945,"number":77,"title":"Documentation of 'group' argument in callbacks passed to each","user":{"login":"tomwanzek","id":16323903,"avatar_url":"https://avatars.githubusercontent.com/u/16323903?v=3","gravatar_id":"","url":"https://api.github.com/users/tomwanzek","html_url":"https://github.com/tomwanzek","followers_url":"https://api.github.com/users/tomwanzek/followers","following_url":"https://api.github.com/users/tomwanzek/following{/other_user}","gists_url":"https://api.github.com/users/tomwanzek/gists{/gist_id}","starred_url":"https://api.github.com/users/tomwanzek/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/tomwanzek/subscriptions","organizations_url":"https://api.github.com/users/tomwanzek/orgs","repos_url":"https://api.github.com/users/tomwanzek/repos","events_url":"https://api.github.com/users/tomwanzek/events{/privacy}","received_events_url":"https://api.github.com/users/tomwanzek/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":0,"created_at":"2016-06-08T14:40:15Z","updated_at":"2016-06-08T14:40:15Z","closed_at":null,"body":"@mbostock, first of all, thanks for the tremendous work on D3.\r\n\r\nI started going through your v4 modules with an eye on updated typescript definition files to go along with them (current DefinitelyTyped definitions are for D3 3.x).\r\n\r\nAs I went through the code of the d3-selection module, I noticed that, the `each(...)` method and by implication the exposed methods which internally use `each` (e.g. `attr(...)`, `style(...)`, `prop(...)`) are all passed the `group` argument.\r\n\r\nBefore making any assumptions, is there a particular intent, why the 'group' argument is not described in the documentation, as is the passing in of the `datum` and `index` arguments?\r\n\r\nAs your library is a fast moving target, thanks to your prolific efforts :smile:, I am currently deciding on the best approach for getting the in-progress updated definitions to the community to co-evolve (pull request on DefinitelyTyped or similar).\r\n\r\nI kept it in a local repo for the past few days, as I am also experimenting with a new typescript@next feature, which allows the typing of the 'this' context within functions such as the ones used as value functions passed into `each(...)` et al. with a DOM element binding.\r\n\r\nThanks in advance.\r\n\r\n"}}
{"actor":{"display_login":"tomwanzek"}}