{"action":"created","issue":{"url":"https://api.github.com/repos/naver/egjs/issues/398","repository_url":"https://api.github.com/repos/naver/egjs","labels_url":"https://api.github.com/repos/naver/egjs/issues/398/labels{/name}","comments_url":"https://api.github.com/repos/naver/egjs/issues/398/comments","events_url":"https://api.github.com/repos/naver/egjs/issues/398/events","html_url":"https://github.com/naver/egjs/pull/398","id":186282293,"number":398,"title":"feat(flicking): Add `dynamicHeight` option","user":{"login":"incleaf","id":7221609,"avatar_url":"https://avatars.githubusercontent.com/u/7221609?v=3","gravatar_id":"","url":"https://api.github.com/users/incleaf","html_url":"https://github.com/incleaf","followers_url":"https://api.github.com/users/incleaf/followers","following_url":"https://api.github.com/users/incleaf/following{/other_user}","gists_url":"https://api.github.com/users/incleaf/gists{/gist_id}","starred_url":"https://api.github.com/users/incleaf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/incleaf/subscriptions","organizations_url":"https://api.github.com/users/incleaf/orgs","repos_url":"https://api.github.com/users/incleaf/repos","events_url":"https://api.github.com/users/incleaf/events{/privacy}","received_events_url":"https://api.github.com/users/incleaf/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":5,"created_at":"2016-10-31T13:24:18Z","updated_at":"2016-11-02T05:28:56Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/naver/egjs/pulls/398","html_url":"https://github.com/naver/egjs/pull/398","diff_url":"https://github.com/naver/egjs/pull/398.diff","patch_url":"https://github.com/naver/egjs/pull/398.patch"},"body":"## Details\r\nI added `dynamicHeight` option that makes container's height to be fitted dynamically from its children's height."},"comment":{"url":"https://api.github.com/repos/naver/egjs/issues/comments/257776200","html_url":"https://github.com/naver/egjs/pull/398#issuecomment-257776200","issue_url":"https://api.github.com/repos/naver/egjs/issues/398","id":257776200,"user":{"login":"incleaf","id":7221609,"avatar_url":"https://avatars.githubusercontent.com/u/7221609?v=3","gravatar_id":"","url":"https://api.github.com/users/incleaf","html_url":"https://github.com/incleaf","followers_url":"https://api.github.com/users/incleaf/followers","following_url":"https://api.github.com/users/incleaf/following{/other_user}","gists_url":"https://api.github.com/users/incleaf/gists{/gist_id}","starred_url":"https://api.github.com/users/incleaf/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/incleaf/subscriptions","organizations_url":"https://api.github.com/users/incleaf/orgs","repos_url":"https://api.github.com/users/incleaf/repos","events_url":"https://api.github.com/users/incleaf/events{/privacy}","received_events_url":"https://api.github.com/users/incleaf/received_events","type":"User","site_admin":false},"created_at":"2016-11-02T05:28:56Z","updated_at":"2016-11-02T05:28:56Z","body":"Thank you for your great comment! I like the way you suggested and it looks good at performance. but I guess we have a different perspective on this option. I thought changing container’s height in every single time would be bad for user experience and also will trigger the repaint. So this is why I fixed every panel’s height in initialization time. I think both ways are needed as an option. What do you think about my opinion?"}}
{"actor":{"display_login":"incleaf"}}