{"action":"created","issue":{"url":"https://api.github.com/repos/emberjs/ember.js/issues/13617","repository_url":"https://api.github.com/repos/emberjs/ember.js","labels_url":"https://api.github.com/repos/emberjs/ember.js/issues/13617/labels{/name}","comments_url":"https://api.github.com/repos/emberjs/ember.js/issues/13617/comments","events_url":"https://api.github.com/repos/emberjs/ember.js/issues/13617/events","html_url":"https://github.com/emberjs/ember.js/pull/13617","id":158965415,"number":13617,"title":"[GLIMMER] Cleanup Glimmer View Support Mixin","user":{"login":"chadhietala","id":183799,"avatar_url":"https://avatars.githubusercontent.com/u/183799?v=3","gravatar_id":"","url":"https://api.github.com/users/chadhietala","html_url":"https://github.com/chadhietala","followers_url":"https://api.github.com/users/chadhietala/followers","following_url":"https://api.github.com/users/chadhietala/following{/other_user}","gists_url":"https://api.github.com/users/chadhietala/gists{/gist_id}","starred_url":"https://api.github.com/users/chadhietala/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/chadhietala/subscriptions","organizations_url":"https://api.github.com/users/chadhietala/orgs","repos_url":"https://api.github.com/users/chadhietala/repos","events_url":"https://api.github.com/users/chadhietala/events{/privacy}","received_events_url":"https://api.github.com/users/chadhietala/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/emberjs/ember.js/labels/Cleanup","name":"Cleanup","color":"5319e7"},{"url":"https://api.github.com/repos/emberjs/ember.js/labels/Glimmer2","name":"Glimmer2","color":"1d76db"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":16,"created_at":"2016-06-07T16:19:02Z","updated_at":"2016-06-09T19:09:16Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/emberjs/ember.js/pulls/13617","html_url":"https://github.com/emberjs/ember.js/pull/13617","diff_url":"https://github.com/emberjs/ember.js/pull/13617.diff","patch_url":"https://github.com/emberjs/ember.js/pull/13617.patch"},"body":"In glimmer we have a better strategy around setting layout and thus\r\nprivate APIS such as `templateNameFor` are no longer needed. This also\r\nremoves the following from the view support mixin.\r\n\r\n- CP for `template` (does nothing today)\r\n- CP for `layout` (cannot change post init, slotted in component class)\r\n- Slotted `layoutName` (in Component class)\r\n- Slotted `templateName` (doesn't work in components and was deprecated)\r\n- Slotted `isView` (not used in glimmer)\r\n\r\nThis also consolidates the component lookup between both engines and removes the `{{view}}` keyword from HTMLBars.\r\n\r\n/cc @krisselden"},"comment":{"url":"https://api.github.com/repos/emberjs/ember.js/issues/comments/224995990","html_url":"https://github.com/emberjs/ember.js/pull/13617#issuecomment-224995990","issue_url":"https://api.github.com/repos/emberjs/ember.js/issues/13617","id":224995990,"user":{"login":"krisselden","id":61024,"avatar_url":"https://avatars.githubusercontent.com/u/61024?v=3","gravatar_id":"","url":"https://api.github.com/users/krisselden","html_url":"https://github.com/krisselden","followers_url":"https://api.github.com/users/krisselden/followers","following_url":"https://api.github.com/users/krisselden/following{/other_user}","gists_url":"https://api.github.com/users/krisselden/gists{/gist_id}","starred_url":"https://api.github.com/users/krisselden/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/krisselden/subscriptions","organizations_url":"https://api.github.com/users/krisselden/orgs","repos_url":"https://api.github.com/users/krisselden/repos","events_url":"https://api.github.com/users/krisselden/events{/privacy}","received_events_url":"https://api.github.com/users/krisselden/received_events","type":"User","site_admin":false},"created_at":"2016-06-09T19:09:16Z","updated_at":"2016-06-09T19:09:16Z","body":"@chadhietala lets not branch where not needed, I'm going to merge, but we should try to get the paths the same as much as possible, so when we improve a path for the glimmer side that isn't actually tied to how glimmer works, we should bring both sides along.  I reduces complexity and confusion.  This forked just simply because glimmer wasn't ready for layout as a prop so it was hacked into the static resolution, if late bound layout had been supported, we wouldn't have forked this."}}
{"actor":{"display_login":"krisselden"}}