summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorSimon Fondrie-Teitler <simonft@riseup.net>2017-05-24 18:47:53 -0400
committerKali Kaneko <kali@leap.se>2017-08-08 18:41:46 -0400
commitbde189698517ea368537974b9878074368797d2f (patch)
treeba47c0a4d9a728dcbb33bc177a82aba9887be9be
parentdffaee85810a92b642d7cda0eb9adfdcaa4cf27a (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.
-rw-r--r--ui/app/components/confirmation.js4
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,