From 20645375642b88f7d525a0fecb424f4587edf41c Mon Sep 17 00:00:00 2001
From: Maxime Beauchemin <maximebeauchemin@gmail.com>
Date: Tue, 18 Apr 2017 12:56:16 -0700
Subject: [PATCH] Addressing comments

---
 superset/assets/javascripts/components/ModalTrigger.jsx       | 3 ---
 superset/assets/javascripts/dashboard/components/Controls.jsx | 1 -
 2 files changed, 4 deletions(-)

diff --git a/superset/assets/javascripts/components/ModalTrigger.jsx b/superset/assets/javascripts/components/ModalTrigger.jsx
index aaff97861b0a..aee93ba82687 100644
--- a/superset/assets/javascripts/components/ModalTrigger.jsx
+++ b/superset/assets/javascripts/components/ModalTrigger.jsx
@@ -15,7 +15,6 @@ const propTypes = {
   bsSize: PropTypes.string,
   className: PropTypes.string,
   tooltip: PropTypes.string,
-  disabled: PropTypes.bool,
 };
 
 const defaultProps = {
@@ -25,7 +24,6 @@ const defaultProps = {
   isButton: false,
   bsSize: null,
   className: '',
-  disabled: false,
 };
 
 export default class ModalTrigger extends React.Component {
@@ -82,7 +80,6 @@ export default class ModalTrigger extends React.Component {
           className="modal-trigger"
           tooltip={this.props.tooltip}
           onClick={this.open}
-          disabled={this.props.disabled}
         >
           {this.props.triggerNode}
           {this.renderModal()}
diff --git a/superset/assets/javascripts/dashboard/components/Controls.jsx b/superset/assets/javascripts/dashboard/components/Controls.jsx
index 75fe1bc0b53b..09b8937e2426 100644
--- a/superset/assets/javascripts/dashboard/components/Controls.jsx
+++ b/superset/assets/javascripts/dashboard/components/Controls.jsx
@@ -95,7 +95,6 @@ class Controls extends React.PureComponent {
         </Button>
         <SaveModal
           dashboard={dashboard}
-          disabled={!dashboard.dash_save_perm}
           css={this.state.css}
           triggerNode={
             <Button disabled={!dashboard.dash_save_perm}>