Skip to content

Commit

Permalink
update Modal.js to fix #10662
Browse files Browse the repository at this point in the history
Summary:
further discussion: should there be a `onClose` or `onClosed` to pair with `onShow`? which would make a workaround for #10471 much easier
Closes #10669

Differential Revision: D4133832

Pulled By: hramos

fbshipit-source-id: 644a5bb6b9da697c81fc96ae4da196ba5b4050cb
  • Loading branch information
nihgwu authored and Facebook Github Bot committed Nov 5, 2016
1 parent 6ed4934 commit 6a83ac3
Showing 1 changed file with 2 additions and 3 deletions.
5 changes: 2 additions & 3 deletions Libraries/Modal/Modal.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,9 +98,8 @@ class Modal extends React.Component {
*/
visible: PropTypes.bool,
/**
* The `onRequestClose` prop allows passing a function that will be called once the modal has been dismissed.
*
* _On the Android platform, this is a required function._
* The `onRequestClose` callback is called when the user taps the hardware back button.
* @platform android
*/
onRequestClose: Platform.OS === 'android' ? PropTypes.func.isRequired : PropTypes.func,
/**
Expand Down

0 comments on commit 6a83ac3

Please sign in to comment.