Skip to content

Commit

Permalink
Addressing comments
Browse files Browse the repository at this point in the history
  • Loading branch information
mistercrunch committed Apr 18, 2017
1 parent 07167fc commit 2064537
Show file tree
Hide file tree
Showing 2 changed files with 0 additions and 4 deletions.
3 changes: 0 additions & 3 deletions superset/assets/javascripts/components/ModalTrigger.jsx
Expand Up @@ -15,7 +15,6 @@ const propTypes = {
bsSize: PropTypes.string,
className: PropTypes.string,
tooltip: PropTypes.string,
disabled: PropTypes.bool,
};

const defaultProps = {
Expand All @@ -25,7 +24,6 @@ const defaultProps = {
isButton: false,
bsSize: null,
className: '',
disabled: false,
};

export default class ModalTrigger extends React.Component {
Expand Down Expand Up @@ -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()}
Expand Down
Expand Up @@ -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}>
Expand Down

0 comments on commit 2064537

Please sign in to comment.