{"action":"created","issue":{"url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/51","labels_url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/51/labels{/name}","comments_url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/51/comments","events_url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/51/events","html_url":"https://github.com/JedWatson/touchstonejs/pull/51","id":62646142,"number":51,"title":"Modal Refactor to CSSTransitionGroup","user":{"login":"KeKs0r","id":2970814,"avatar_url":"https://avatars.githubusercontent.com/u/2970814?v=3","gravatar_id":"","url":"https://api.github.com/users/KeKs0r","html_url":"https://github.com/KeKs0r","followers_url":"https://api.github.com/users/KeKs0r/followers","following_url":"https://api.github.com/users/KeKs0r/following{/other_user}","gists_url":"https://api.github.com/users/KeKs0r/gists{/gist_id}","starred_url":"https://api.github.com/users/KeKs0r/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KeKs0r/subscriptions","organizations_url":"https://api.github.com/users/KeKs0r/orgs","repos_url":"https://api.github.com/users/KeKs0r/repos","events_url":"https://api.github.com/users/KeKs0r/events{/privacy}","received_events_url":"https://api.github.com/users/KeKs0r/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":1,"created_at":"2015-03-18T10:05:29Z","updated_at":"2015-04-11T11:52:52Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/JedWatson/touchstonejs/pulls/51","html_url":"https://github.com/JedWatson/touchstonejs/pull/51","diff_url":"https://github.com/JedWatson/touchstonejs/pull/51.diff","patch_url":"https://github.com/JedWatson/touchstonejs/pull/51.patch"},"body":"I wanted to use the Modal, but in its current version somehow it was always visible, since the visibility depended on showModal state, which was automatically.\r\n\r\nI have also refactored it to ReactCSSTransitionGroup for Animations. In order to access the default animations of Touchstone I have taken them from \"createApp\" into an extra mixin, which can now be used by anyone in any other component, if they want to \"animate\" things with touchstone animations.\r\n\r\nI also added the possibility to provide the content of the Modal as child props, which should make it especially for more complex HTML or JSX more comfortable to provide content to the modal."},"comment":{"url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/comments/91832418","html_url":"https://github.com/JedWatson/touchstonejs/pull/51#issuecomment-91832418","issue_url":"https://api.github.com/repos/JedWatson/touchstonejs/issues/51","id":91832418,"user":{"login":"KeKs0r","id":2970814,"avatar_url":"https://avatars.githubusercontent.com/u/2970814?v=3","gravatar_id":"","url":"https://api.github.com/users/KeKs0r","html_url":"https://github.com/KeKs0r","followers_url":"https://api.github.com/users/KeKs0r/followers","following_url":"https://api.github.com/users/KeKs0r/following{/other_user}","gists_url":"https://api.github.com/users/KeKs0r/gists{/gist_id}","starred_url":"https://api.github.com/users/KeKs0r/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/KeKs0r/subscriptions","organizations_url":"https://api.github.com/users/KeKs0r/orgs","repos_url":"https://api.github.com/users/KeKs0r/repos","events_url":"https://api.github.com/users/KeKs0r/events{/privacy}","received_events_url":"https://api.github.com/users/KeKs0r/received_events","type":"User","site_admin":false},"created_at":"2015-04-11T11:52:52Z","updated_at":"2015-04-11T11:52:52Z","body":"Any Feedback on this? If you don't like the animation things I can also solve it differently but currently the modals are somewhat \"unusable\" because they always show and you cant define easily more complex content (which would be done via the \"children\" props)"}}