{"action":"created","issue":{"url":"https://api.github.com/repos/angular/angular/issues/5638","labels_url":"https://api.github.com/repos/angular/angular/issues/5638/labels{/name}","comments_url":"https://api.github.com/repos/angular/angular/issues/5638/comments","events_url":"https://api.github.com/repos/angular/angular/issues/5638/events","html_url":"https://github.com/angular/angular/issues/5638","id":120572877,"number":5638,"title":"alpha.48: tbody should not wrap tr in template","user":{"login":"todoubaba","id":4020640,"avatar_url":"https://avatars.githubusercontent.com/u/4020640?v=3","gravatar_id":"","url":"https://api.github.com/users/todoubaba","html_url":"https://github.com/todoubaba","followers_url":"https://api.github.com/users/todoubaba/followers","following_url":"https://api.github.com/users/todoubaba/following{/other_user}","gists_url":"https://api.github.com/users/todoubaba/gists{/gist_id}","starred_url":"https://api.github.com/users/todoubaba/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/todoubaba/subscriptions","organizations_url":"https://api.github.com/users/todoubaba/orgs","repos_url":"https://api.github.com/users/todoubaba/repos","events_url":"https://api.github.com/users/todoubaba/events{/privacy}","received_events_url":"https://api.github.com/users/todoubaba/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/angular/angular/labels/type:%20bug","name":"type: bug","color":"EEEEEE"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/angular/angular/milestones/49","html_url":"https://github.com/angular/angular/milestones/beta.0","labels_url":"https://api.github.com/repos/angular/angular/milestones/49/labels","id":1339148,"number":49,"title":"beta.0","description":"Issues which must close before we declare beta.","creator":{"login":"naomiblack","id":2286589,"avatar_url":"https://avatars.githubusercontent.com/u/2286589?v=3","gravatar_id":"","url":"https://api.github.com/users/naomiblack","html_url":"https://github.com/naomiblack","followers_url":"https://api.github.com/users/naomiblack/followers","following_url":"https://api.github.com/users/naomiblack/following{/other_user}","gists_url":"https://api.github.com/users/naomiblack/gists{/gist_id}","starred_url":"https://api.github.com/users/naomiblack/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/naomiblack/subscriptions","organizations_url":"https://api.github.com/users/naomiblack/orgs","repos_url":"https://api.github.com/users/naomiblack/repos","events_url":"https://api.github.com/users/naomiblack/events{/privacy}","received_events_url":"https://api.github.com/users/naomiblack/received_events","type":"User","site_admin":false},"open_issues":56,"closed_issues":143,"state":"open","created_at":"2015-10-05T21:32:20Z","updated_at":"2015-12-06T21:23:27Z","due_on":null,"closed_at":null},"comments":2,"created_at":"2015-12-05T17:13:39Z","updated_at":"2015-12-06T22:19:19Z","closed_at":null,"body":"```typescript\r\n@Component({\r\n  selector: 'my-cmp',\r\n  template: `\r\n  <table border>\r\n    <thead>\r\n      <th>col1</th>\r\n      <th>col2</th>\r\n    </thead>\r\n    <tbody>\r\n      <template ng-for #rx [ng-for-of]=\"values\">\r\n        <tr>\r\n          <template ng-for #ry [ng-for-of]=\"rx\">\r\n            <td>{{ry}}</td>\r\n          </template>\r\n        </tr>\r\n      </template>\r\n    </tbody>\r\n  </table>\r\n  `\r\n})\r\nclass MyCmp {\r\n  values = [\r\n    [11, 12],\r\n    [21, 22]\r\n  ];\r\n}\r\n```\r\n\r\n![table](http://qpic.cn/SwWR93zOa)\r\n![Elements](http://qpic.cn/ikyiNsRQE)\r\n\r\nhttp://plnkr.co/edit/ZXV6pLI44IqFQbIEX4zj?p=preview\r\n"},"comment":{"url":"https://api.github.com/repos/angular/angular/issues/comments/162353879","html_url":"https://github.com/angular/angular/issues/5638#issuecomment-162353879","issue_url":"https://api.github.com/repos/angular/angular/issues/5638","id":162353879,"user":{"login":"vicb","id":248818,"avatar_url":"https://avatars.githubusercontent.com/u/248818?v=3","gravatar_id":"","url":"https://api.github.com/users/vicb","html_url":"https://github.com/vicb","followers_url":"https://api.github.com/users/vicb/followers","following_url":"https://api.github.com/users/vicb/following{/other_user}","gists_url":"https://api.github.com/users/vicb/gists{/gist_id}","starred_url":"https://api.github.com/users/vicb/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/vicb/subscriptions","organizations_url":"https://api.github.com/users/vicb/orgs","repos_url":"https://api.github.com/users/vicb/repos","events_url":"https://api.github.com/users/vicb/events{/privacy}","received_events_url":"https://api.github.com/users/vicb/received_events","type":"User","site_admin":false},"created_at":"2015-12-06T22:19:18Z","updated_at":"2015-12-06T22:19:18Z","body":"Two way we can fix this:\r\n- never explicitly add a `<tbody>`,\r\n- add a `<tbody>` only if the `<tr>` has a parent (ie is not a fragment).\r\n\r\nThe first solution would produce consistent result whether rows are in the master template or added as fragment.\r\n\r\n@tbosch what do you think ?"}}