{"action":"created","issue":{"url":"https://api.github.com/repos/react-community/react-navigation/issues/232","repository_url":"https://api.github.com/repos/react-community/react-navigation","labels_url":"https://api.github.com/repos/react-community/react-navigation/issues/232/labels{/name}","comments_url":"https://api.github.com/repos/react-community/react-navigation/issues/232/comments","events_url":"https://api.github.com/repos/react-community/react-navigation/issues/232/events","html_url":"https://github.com/react-community/react-navigation/issues/232","id":206204060,"number":232,"title":"mode prop for Screen","user":{"login":"nihgwu","id":2595058,"avatar_url":"https://avatars0.githubusercontent.com/u/2595058?v=3","gravatar_id":"","url":"https://api.github.com/users/nihgwu","html_url":"https://github.com/nihgwu","followers_url":"https://api.github.com/users/nihgwu/followers","following_url":"https://api.github.com/users/nihgwu/following{/other_user}","gists_url":"https://api.github.com/users/nihgwu/gists{/gist_id}","starred_url":"https://api.github.com/users/nihgwu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/nihgwu/subscriptions","organizations_url":"https://api.github.com/users/nihgwu/orgs","repos_url":"https://api.github.com/users/nihgwu/repos","events_url":"https://api.github.com/users/nihgwu/events{/privacy}","received_events_url":"https://api.github.com/users/nihgwu/received_events","type":"User","site_admin":false},"labels":[{"id":526742650,"url":"https://api.github.com/repos/react-community/react-navigation/labels/feature%20request","name":"feature request","color":"1abc9c","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2017-02-08T13:53:32Z","updated_at":"2017-04-13T01:56:18Z","closed_at":"2017-04-13T01:56:18Z","body":"Right now `mode` is only available for `StackNavigatorConfig`and only accepts a string, so all the `Screen`s in it are with the same transition mode, if we want different modes between views, we need to define different `Navigators` and nest them, which makes things complex, so I have the following propose:\r\n1. add `mode` to `Screen Navigation Options`\r\n2. make `mode` accepts a function\r\n\r\nAny ideas?  I'd love to send a PR to implement\r\n"},"comment":{"url":"https://api.github.com/repos/react-community/react-navigation/issues/comments/293755383","html_url":"https://github.com/react-community/react-navigation/issues/232#issuecomment-293755383","issue_url":"https://api.github.com/repos/react-community/react-navigation/issues/232","id":293755383,"user":{"login":"ericvicenti","id":1483597,"avatar_url":"https://avatars2.githubusercontent.com/u/1483597?v=3","gravatar_id":"","url":"https://api.github.com/users/ericvicenti","html_url":"https://github.com/ericvicenti","followers_url":"https://api.github.com/users/ericvicenti/followers","following_url":"https://api.github.com/users/ericvicenti/following{/other_user}","gists_url":"https://api.github.com/users/ericvicenti/gists{/gist_id}","starred_url":"https://api.github.com/users/ericvicenti/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ericvicenti/subscriptions","organizations_url":"https://api.github.com/users/ericvicenti/orgs","repos_url":"https://api.github.com/users/ericvicenti/repos","events_url":"https://api.github.com/users/ericvicenti/events{/privacy}","received_events_url":"https://api.github.com/users/ericvicenti/received_events","type":"User","site_admin":false},"created_at":"2017-04-13T01:56:17Z","updated_at":"2017-04-13T01:56:17Z","body":"For now, you can use this hack to easily support both types of screen inside of one navigator:\r\n\r\nhttps://github.com/react-community/react-navigation/issues/707#issuecomment-293748852\r\n\r\nIt would be a huge change to add this feature as a navigation option, and we are trying to stabilize for v1. @nihgwu, if you have an idea for a simple implementation that doesn't cause lots of regressions, feel free to open a new issue and tag me."}}
{"actor":{"display_login":"ericvicenti"}}