mirror of
https://github.com/facebook/react-native.git
synced 2025-11-01 09:14:26 +00:00
Paige SunandFacebook GitHub Bot
7bf78eae5e
Summary: Changelog: [iOS] Use visible prop to dismiss Modal on old renderer. Visible prop is used on Fabric so that onDismiss can be passed with the the bridgeless per-component event emitter, rather than the bridge global event emitter. The old renderer still uses the global event emitter. I needed to use the visible prop for Paper too because in diff 6/7, Modal.js [no longer uses visible prop to return null](https://github.com/facebook/react-native/blob/dc80b2dcb52fadec6a573a9dd1824393f8c29fdc/Libraries/Modal/Modal.js#L221-L222), and Modal.js can't distinguish on whether it's a Fabric or Paper component. Reviewed By: JoshuaGross Differential Revision: D28137929 fbshipit-source-id: f6ede0019fbe498a10b822ff09fc135a9fff8ec0