{"action":"created","issue":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/6174","repository_url":"https://api.github.com/repos/Automattic/wp-calypso","labels_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/6174/labels{/name}","comments_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/6174/comments","events_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/6174/events","html_url":"https://github.com/Automattic/wp-calypso/pull/6174","id":161440922,"number":6174,"title":"[WIP] Stats \"First View\"","user":{"login":"mattsherman","id":2098816,"avatar_url":"https://avatars.githubusercontent.com/u/2098816?v=3","gravatar_id":"","url":"https://api.github.com/users/mattsherman","html_url":"https://github.com/mattsherman","followers_url":"https://api.github.com/users/mattsherman/followers","following_url":"https://api.github.com/users/mattsherman/following{/other_user}","gists_url":"https://api.github.com/users/mattsherman/gists{/gist_id}","starred_url":"https://api.github.com/users/mattsherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattsherman/subscriptions","organizations_url":"https://api.github.com/users/mattsherman/orgs","repos_url":"https://api.github.com/users/mattsherman/repos","events_url":"https://api.github.com/users/mattsherman/events{/privacy}","received_events_url":"https://api.github.com/users/mattsherman/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/NUX","name":"NUX","color":"006b75"},{"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/Stats","name":"Stats","color":"006b75"},{"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Status]%20In%20Progress","name":"[Status] In Progress","color":"207de5"},{"url":"https://api.github.com/repos/Automattic/wp-calypso/labels/[Type]%20Task","name":"[Type] Task","color":"0052cc"}],"state":"open","locked":false,"assignee":{"login":"mattsherman","id":2098816,"avatar_url":"https://avatars.githubusercontent.com/u/2098816?v=3","gravatar_id":"","url":"https://api.github.com/users/mattsherman","html_url":"https://github.com/mattsherman","followers_url":"https://api.github.com/users/mattsherman/followers","following_url":"https://api.github.com/users/mattsherman/following{/other_user}","gists_url":"https://api.github.com/users/mattsherman/gists{/gist_id}","starred_url":"https://api.github.com/users/mattsherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattsherman/subscriptions","organizations_url":"https://api.github.com/users/mattsherman/orgs","repos_url":"https://api.github.com/users/mattsherman/repos","events_url":"https://api.github.com/users/mattsherman/events{/privacy}","received_events_url":"https://api.github.com/users/mattsherman/received_events","type":"User","site_admin":false},"assignees":[{"login":"mattsherman","id":2098816,"avatar_url":"https://avatars.githubusercontent.com/u/2098816?v=3","gravatar_id":"","url":"https://api.github.com/users/mattsherman","html_url":"https://github.com/mattsherman","followers_url":"https://api.github.com/users/mattsherman/followers","following_url":"https://api.github.com/users/mattsherman/following{/other_user}","gists_url":"https://api.github.com/users/mattsherman/gists{/gist_id}","starred_url":"https://api.github.com/users/mattsherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattsherman/subscriptions","organizations_url":"https://api.github.com/users/mattsherman/orgs","repos_url":"https://api.github.com/users/mattsherman/repos","events_url":"https://api.github.com/users/mattsherman/events{/privacy}","received_events_url":"https://api.github.com/users/mattsherman/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2016-06-21T13:45:43Z","updated_at":"2016-06-25T14:19:04Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/Automattic/wp-calypso/pulls/6174","html_url":"https://github.com/Automattic/wp-calypso/pull/6174","diff_url":"https://github.com/Automattic/wp-calypso/pull/6174.diff","patch_url":"https://github.com/Automattic/wp-calypso/pull/6174.patch"},"body":"**WIP: Not ready for review or merging yet.**\r\n\r\n\n\nTest live: https://calypso.live/?branch=add/stats-first-view-ui"},"comment":{"url":"https://api.github.com/repos/Automattic/wp-calypso/issues/comments/228545273","html_url":"https://github.com/Automattic/wp-calypso/pull/6174#issuecomment-228545273","issue_url":"https://api.github.com/repos/Automattic/wp-calypso/issues/6174","id":228545273,"user":{"login":"mattsherman","id":2098816,"avatar_url":"https://avatars.githubusercontent.com/u/2098816?v=3","gravatar_id":"","url":"https://api.github.com/users/mattsherman","html_url":"https://github.com/mattsherman","followers_url":"https://api.github.com/users/mattsherman/followers","following_url":"https://api.github.com/users/mattsherman/following{/other_user}","gists_url":"https://api.github.com/users/mattsherman/gists{/gist_id}","starred_url":"https://api.github.com/users/mattsherman/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/mattsherman/subscriptions","organizations_url":"https://api.github.com/users/mattsherman/orgs","repos_url":"https://api.github.com/users/mattsherman/repos","events_url":"https://api.github.com/users/mattsherman/events{/privacy}","received_events_url":"https://api.github.com/users/mattsherman/received_events","type":"User","site_admin":false},"created_at":"2016-06-25T14:19:04Z","updated_at":"2016-06-25T14:19:04Z","body":"@mtias The main problem I'm trying to tackle is how to provide a component that handles the animation for both the `FirstView` and the `Main` component that the `FirstView` is a 'child' of. When the `FirstView` is shown, I also need to set CSS classes on the `Main` component in order to blur and scale it. And I want to provide this capability to a number of different `Main` components in the future (we're starting with stats, but other sections will get first views eventually). That's why I felt a wrapper component was necessary.\r\n\r\nI wasn't aware of the \"higher order component\" approach. Is this what you are talking about? http://www.natpryce.com/articles/000814.html\r\n\r\nDo you have any examples in Calypso that I should use as reference?\r\n\r\nThat seems like a clearer approach that still wraps things but avoids the noise you were concerned about. I will change things to take this approach.\r\n\r\nThanks for helping to make sure this PR does things the most appropriate way for Calypso -- I really appreciate the early feedback."}}
{"actor":{"display_login":"mattsherman"}}