{"push_id":1239606656,"size":52,"distinct_size":52,"ref":"refs/heads/scrollview-snap-bug-fix","head":"d457321ae08344dcec171abd710601e2e038f2e9","before":"c5b2154ad0b89ac99942306e29e1a09ba5bc4577","commits":[{"sha":"063f221e5671d53af0e2b5e4b0dda555bd8ddefd","author":{"name":"G. Grau","email":"c4fb8ca08597468ef97d388c991d8a49072d625b@gmail.com"},"message":"UIExplorer: fix navigation animation by using props passed to renderTitleComponent\n\nSummary:\nThe animation shown in the UIExplorer example for NavigationExperimental's NavigationHeader is not working correctly. Due to an incomplete use of the `renderTitleComponent()` callback, interpolation is not working correctly in this example.\n\nEven if the animation error is subtle, since the examples are followed by developers quite closely, the bug may multiply if not corrected in the sample code.\n\nThis is a really *tiny* modification that fixes that particular animation.\nCloses https://github.com/facebook/react-native/pull/9067\n\nDifferential Revision: D3669140\n\nPulled By: javache\n\nfbshipit-source-id: 6dab36a9132da2f704e9a074ae26e744c3420cde","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/063f221e5671d53af0e2b5e4b0dda555bd8ddefd"},{"sha":"523102b8e8d82ffd0ee12b1778a6aa74a6046d56","author":{"name":"Melih Mucuk","email":"99e9569824d324ceb2ad654f1ef3c384937d9ed7@gmail.com"},"message":"Add melihmucuk to Github Issue Task Force\n\nSummary:\nHere we go !\nCloses https://github.com/facebook/react-native/pull/9143\n\nDifferential Revision: D3669363\n\nPulled By: mkonicek\n\nfbshipit-source-id: 7fbd16be25736055bd2d84e34ad7e66f0d885a0a","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/523102b8e8d82ffd0ee12b1778a6aa74a6046d56"},{"sha":"223bcd74ad7a474e63d89a0cc882dd4a747caafe","author":{"name":"wenzhao.yin","email":"a1e2e3da0fe0f013fcd9fc6b3894baa62ad66acf@qunar.com"},"message":"Cancel check update task\n\nSummary:\nWhen call `[RCTDevMenu invalidate]`, it will cancel updateTask. But in fact, after the `[updateTask cancel]`, the updateTask's completionHandler will call `checkForUpdates` again, so it will create a new updateTask.\nCloses https://github.com/facebook/react-native/pull/6165\n\nDifferential Revision: D3669576\n\nPulled By: javache\n\nfbshipit-source-id: 443924b7f4be1716797fc5690e90d962cf31b923","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/223bcd74ad7a474e63d89a0cc882dd4a747caafe"},{"sha":"5fd6c0903ecfd271158095fb8ded1d8156438ce5","author":{"name":"Emil Sjolander","email":"fd01956b6f7ea3f89c72dc9353cbe274f619cc70@fb.com"},"message":"Add jni bindings\n\nReviewed By: lucasr\n\nDifferential Revision: D3648793\n\nfbshipit-source-id: d0e696e196fa7c63109c9117a65645ca3d6c9c00","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/5fd6c0903ecfd271158095fb8ded1d8156438ce5"},{"sha":"44e7a88620427efc93896b6b76b58c448a15f9e5","author":{"name":"Emil Sjolander","email":"fd01956b6f7ea3f89c72dc9353cbe274f619cc70@fb.com"},"message":"Fallback to System.loadLibrary if SoLoader has not been initialized\n\nReviewed By: lucasr\n\nDifferential Revision: D3661990\n\nfbshipit-source-id: f2003577aa3d2f89ec579b6f889fdfb684110b60","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/44e7a88620427efc93896b6b76b58c448a15f9e5"},{"sha":"e0fc8c0ea03c6f0b24dd1d23adb84ac522c397ed","author":{"name":"Emil Sjolander","email":"fd01956b6f7ea3f89c72dc9353cbe274f619cc70@fb.com"},"message":"Add the ability to attach data to a java CSSNode\n\nReviewed By: lucasr\n\nDifferential Revision: D3662065\n\nfbshipit-source-id: 560a768092f17381e99b349d08bd4a8b365541be","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/e0fc8c0ea03c6f0b24dd1d23adb84ac522c397ed"},{"sha":"4983a58d6c739ca9fe7dee323907e39d085056a0","author":{"name":"Emil Sjolander","email":"fd01956b6f7ea3f89c72dc9353cbe274f619cc70@fb.com"},"message":"Add and run clang format script\n\nReviewed By: lucasr\n\nDifferential Revision: D3662225\n\nfbshipit-source-id: ddd4064cbf9be21ca6a97001ace1b56b4314c86f","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/4983a58d6c739ca9fe7dee323907e39d085056a0"},{"sha":"77e48f17824870d30144a583be77ec5c9cf9f8c5","author":{"name":"Dotan Nahum","email":"21576a6e966c52984f5814460eb3ff437922afb7@gmail.com"},"message":"Make MessageQueue to emit \"SPY\" events in a way that can be extensible\n\nSummary:\nThis PR adds a capability for MessageQueue to emit \"SPY\" events in a way that can be extensible, to later allow for a tooling ecosystem to grow, one example is the existing [Snoopy](https://github.com/jondot/rn-snoopy) tool that is, for now, forced to work with monkeypatches, and after this PR will be able to use a \"formal\" way to trace queue events.\n\nAfter this change, we can wire a \"spy\" into a queue that will expose the events in different and interesting ways, see below (done with Snoopy):\n  <img src=\"https://github.com/jondot/rn-snoopy/blob/master/media/snoopy.gif?raw=true\" alt=\"Aggregating and Charting Events with Bar\" width=\"400px\"/>\n  <img src=\"https://github.com/jondot/rn-snoopy/blob/master/media/snoopy-filter.gif?raw=true\" alt=\"Aggregating and Charting Events with Bar\" width=\"400px\"/>\n\nThis removes the hardcoded `SPY_MODE` flag and instead uses a function that can be injected from outside world.\n\n```javascript\nMessageQueue.spy((info)=>console.log(\"event!\", info)\n```\n\nIt also creates\nCloses https://github.com/facebook/react-native/pull/9160\n\nDifferential Revision: D3669053\n\nPulled By: javache\n\nfbshipit-source-id: 3e4462aa77fc8514d2ea4f15430f7bec57b583a4","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/77e48f17824870d30144a583be77ec5c9cf9f8c5"},{"sha":"1d980188f808e36263dbff4938f576c25a9bddcd","author":{"name":"sam","email":"46a37fb0af093dc2faa629ffae8e89b072d3c43b@gmail.com"},"message":"clean UIExplorer NavigationExperimental example code\n\nSummary:\n1. Explain the **motivation** for making this change.\n\n    Those codes cleaned are not used anywhere, remove them would make the example much more clear to users.\n\n2. **Test plan (required)**\n\n    ![navigationexperimental](https://cloud.githubusercontent.com/assets/1091472/17392196/4f28cba4-5a4e-11e6-9ef1-727edc784f1f.gif)\nCloses https://github.com/facebook/react-native/pull/9203\n\nDifferential Revision: D3668848\n\nfbshipit-source-id: c604a21c84dba72a5a3c857fc16bc7febfe20377","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/1d980188f808e36263dbff4938f576c25a9bddcd"},{"sha":"ca8531105e29f2204e0c997783e71b214390bedf","author":{"name":"Hedger Wang","email":"997d4010f1c98bf1d02cd1a42ab2ebf0bf5aa47e@fb.com"},"message":"NavigationExperimental: Rename `renderOverlay` to `renderHeader`\n\nSummary:\nNavigationCardStack is a custom component, and its API should be explicit, not\ntoo generic..\n\nIn NavigationCardStack, the prop `renderOverlay` is actually used to render\nthe NavigationHeader, and we uses absolute position to build the layout for\nthe header and the body.\n\nOne of the problem with using absolute postion and fixed height to build the\nlayout that contains the header is that the header can't have variant height\neasily.\n\nIdeally, if the layout for the header used flex-box, we'd ve able to be more\nadaptive to deal with the header that has variant height.\n\nThat said, let's rename `renderOverlay` to `renderHeader`, then build the\nproper layout that explicitly works better with the header.\n\nIf we to need to support overlay in navigation, we may consider add\n`renderOverlay` later, if it's really necessary.\n\nReviewed By: ericvicenti\n\nDifferential Revision: D3670224\n\nfbshipit-source-id: ff04acfe9dc995cb57117b3fd9b07d5f97b9c6ee","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/ca8531105e29f2204e0c997783e71b214390bedf"},{"sha":"5d0131d31b6d3f61127b6f34da789c3424291459","author":{"name":"Marc Horowitz","email":"5e85808dffeff32aa7915794d93d1e0510e0c781@fb.com"},"message":"Android instrumentation tests for C++ module sync methods.\n\nDifferential Revision: D3574896\n\nfbshipit-source-id: 8cd4c55b7a472c77cc86fd5cbfe43bc4541ba6ba","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/5d0131d31b6d3f61127b6f34da789c3424291459"},{"sha":"341b29da712f40f175378258b35f7cb0b0ad8623","author":{"name":"Kyle Corbitt","email":"d0ff38e6f73256704c61d16b0f8fbefe41e0c8dc@corbt.com"},"message":"Render NavigationHeader correctly with hidden status bar\n\nSummary:\nCurrently, the NavigationExperimental `Header` only renders correctly on iOS when the system status bar is visible. There are legitimate reasons to hide the status bar, especially when displaying in landscape.\n\nThis PR adds a `statusBarHeight` prop to the header, which defaults to 20 on iOS and 0 (no status bar) on Android. Changing this value causes the extra space at the top of the header reserved for the status bar to change.\n\nI've tested this change in my own app on iOS with `statusBarHeight` set to 0, 20, and `undefined`, and ensured that it works correctly.\nCloses https://github.com/facebook/react-native/pull/8983\n\nDifferential Revision: D3668637\n\nfbshipit-source-id: 777a0c53e8fd1caa35ce4980ca3118adcf83b62d","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/341b29da712f40f175378258b35f7cb0b0ad8623"},{"sha":"0dd93b62b4d1f6e570274eb7b3d7f5f557a3ea20","author":{"name":"Spencer Ahrens","email":"45f65c7ce0f9d3e3f549b1244e8bc724fa0484ec@fb.com"},"message":"fix console.table polyfill when entries are missing\n\nSummary: Looks like react-addons-perf is sometimes missing entries. This prevents it from crashing.\n\nReviewed By: davidaurelio\n\nDifferential Revision: D3669007\n\nfbshipit-source-id: 18a1102b5ad8dcfd9b80d39e1172ba85ad0e9dd2","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/0dd93b62b4d1f6e570274eb7b3d7f5f557a3ea20"},{"sha":"f0fb228ec76ed49e6ed6d786d888e8113b8959a2","author":{"name":"Mengjue Wang","email":"860cc5dd44084f7f633d2fba5bf93305f6f0e8c8@fb.com"},"message":"Provide forceRTL for LTR language to test\n\nSummary:\n1. Provide forceRTL function for developer to test RTL layout in LTR language bundle in I18nUtil and expose it in I18nManager.\n2. Rename `allowRTL` and `setAllowRTL` functions\n\nReviewed By: fkgozali\n\nDifferential Revision: D3663693\n\nfbshipit-source-id: 3db13a44c069ae73d1728c211306422db5dd8122","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/f0fb228ec76ed49e6ed6d786d888e8113b8959a2"},{"sha":"2f1af09a36bbcb0ac0b83882fb371a71617a76ed","author":{"name":"Emil Sjolander","email":"fd01956b6f7ea3f89c72dc9353cbe274f619cc70@fb.com"},"message":"Add new csslayout deps to buck file\n\nSummary: Some new csslayout deps were added that were not properly synced to open source.\n\nReviewed By: bestander\n\nDifferential Revision: D3670567\n\nfbshipit-source-id: 6890aabd2eff9dadf9b33a8b44bedc6947a8525e","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/2f1af09a36bbcb0ac0b83882fb371a71617a76ed"},{"sha":"e3c6d8c93132741f802133d14d3240aa07d8b055","author":{"name":"Eric Vicenti","email":"60993fd2931bccf8b94630fa7735ef9356a373c3@fb.com"},"message":"Fix UIExplorer iOS swipe back\n\nReviewed By: hedgerwang\n\nDifferential Revision: D3670396\n\nfbshipit-source-id: 76aafa951d869227f32c2d4c585bb1513e5eea37","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/e3c6d8c93132741f802133d14d3240aa07d8b055"},{"sha":"30677e7193221273295151ea998426b135988498","author":{"name":"Forrest Bice","email":"8691bdc36525aa339f2bc4693f89e917dc23c10a@gmail.com"},"message":"ReactInstanceManager API Updates\n\nSummary:\nUpdating code example with updated method names...\n\n`onHostPause` -> `onPause`\n`onHostResume` -> `onResume`\n`onHostDestroy` -> `onDestroy`\nCloses https://github.com/facebook/react-native/pull/9217\n\nDifferential Revision: D3671267\n\nPulled By: javache\n\nfbshipit-source-id: 007a2b0909fee4495a98c141bd6ac3a564b50c17","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/30677e7193221273295151ea998426b135988498"},{"sha":"158d435f36b9d1ae3637b6df680b4b9abd5e89a5","author":{"name":"Janic Duplessis","email":"f86e95ed6b436aea8f9b2714b1e8fe379ef2d649@gmail.com"},"message":"Implement native Animated value listeners on Android\n\nSummary:\nAdds support for `Animated.Value#addListener` for native driven nodes on Android. This is based on work by skevy in the exponent RN fork. Also adds a UIExplorer example.\n\n** Test plan **\nRun unit tests\n\nTested that by adding a listener to a native driven animated node and checked that the listener callback is called properly.\n\nAlso tested that it doesn't crash on iOS that doesn't support this yet.\nCloses https://github.com/facebook/react-native/pull/8844\n\nDifferential Revision: D3670906\n\nfbshipit-source-id: 15700ed7b93db140d907ce80af4dae6be3102135","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/158d435f36b9d1ae3637b6df680b4b9abd5e89a5"},{"sha":"38979f9c68bd054aca3c6102635b2005914506b1","author":{"name":"Hedger Wang","email":"997d4010f1c98bf1d02cd1a42ab2ebf0bf5aa47e@fb.com"},"message":"NavigationExperimental: Stop using absolute position for NavigationHeader.\n\nSummary:\nNot a API change, but this may break the layout of exisitng apps that\nuses NavigationHeader.\n\nFor now, NavigationHeader uses absolute position, which makes it hard for\nNavigationCardStack to determine the height of the scenes.\n\nTheoretically, the height of the scenes would be the height of the cards\nstack minus the height of the header.\n\nThat said, if we want to support the headers with different height (e.g.\nMyIOSHeader or MyAndroidHeader), we're forced to expose the height of the\nheaders and manually compute the height of the scenes.\n\nAlternatively, if the header does not use absolute position, the height\nof the scenes can adjust automatically with flex box, and that's what this\ncommit is about to do.\n\nReviewed By: ericvicenti\n\nDifferential Revision: D3671119\n\nfbshipit-source-id: 26e48f801da3661c5d7dce4752ba927621172f4a","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/38979f9c68bd054aca3c6102635b2005914506b1"},{"sha":"f08f23f8cb7bb653054968e2c6409281e73f8f83","author":{"name":"Michał Gregorczyk","email":"567e50b999e2a284766a7968a0f9ad1e581d075b@fb.com"},"message":"Add API to set load flags used by UnpackingJSBundleLoader\n\nReviewed By: tadeuzagallo\n\nDifferential Revision: D3557667\n\nfbshipit-source-id: 5665667185cda10415210efce83685c3e3abee92","distinct":true,"url":"https://api.github.com/repos/anttimo/react-native/commits/f08f23f8cb7bb653054968e2c6409281e73f8f83"}]}
{"actor":{"display_login":"anttimo"}}