diff options
| author | Simon Fondrie-Teitler <simonft@riseup.net> | 2017-05-24 18:47:53 -0400 |
|---|---|---|
| committer | Kali Kaneko <kali@leap.se> | 2017-08-08 18:41:46 -0400 |
| commit | bde189698517ea368537974b9878074368797d2f (patch) | |
| tree | ba47c0a4d9a728dcbb33bc177a82aba9887be9be /ui/app/components | |
| parent | dffaee85810a92b642d7cda0eb9adfdcaa4cf27a (diff) | |
[bug] Add "show" option to modal in confirmation component
Defaults to True. Without it, the confirmation modal doesn't show and
you aren't able to remove a provider from the list.
Diffstat (limited to 'ui/app/components')
| -rw-r--r-- | ui/app/components/confirmation.js | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/ui/app/components/confirmation.js b/ui/app/components/confirmation.js index deceb47b..46f69e29 100644 --- a/ui/app/components/confirmation.js +++ b/ui/app/components/confirmation.js @@ -7,7 +7,7 @@ import {Button, ButtonToolbar, Modal} from 'react-bootstrap' import PropTypes from 'prop-types' const Confirmation = props => ( - <Modal> + <Modal show={props.show}> <Modal.Header closeButton> <Modal.Title> {props.title} @@ -27,6 +27,7 @@ const Confirmation = props => ( ) Confirmation.defaultProps = { + show: true, title: "Are you sure?", onCancel: null, onAccept: null, @@ -35,6 +36,7 @@ Confirmation.defaultProps = { } Confirmation.propTypes = { + show: PropTypes.bool, className: PropTypes.string, onCancel: PropTypes.func.isRequired, onAccept: PropTypes.func.isRequired, |