Disable/enable summary clear correctly

aaimio · aaimio · commit 3997850cef11 · 2020-06-17T15:49:05.000+08:00
diff --git a/src/botPage/view/TradeInfoPanel/ClearButton.js b/src/botPage/view/TradeInfoPanel/ClearButton.js
@@ -11,6 +11,7 @@ export default class ClearButton extends React.PureComponent {
     componentDidMount() {
         globalObserver.register('summary.enable_clear', () => this.setState({ isButtonDisabled: false }));
         globalObserver.register('summary.disable_clear', () => this.setState({ isButtonDisabled: true }));
+        globalObserver.register('bot.running', () => this.setState({ isButtonDisabled: true }));
     }
     // eslint-disable-next-line class-methods-use-this
     confirmClearLog() {
diff --git a/src/botPage/view/TradeInfoPanel/index.js b/src/botPage/view/TradeInfoPanel/index.js
@@ -67,6 +67,7 @@ class AnimateTrade extends Component {
                 indicatorMessage: this.indicatorMessages.starting,
                 stopMessage     : this.indicatorMessages.stopped,
             });
+            globalObserver.emit('summary.disable_clear');
             globalObserver.register('contract.status', contractStatus => {
                 this.animateStage(contractStatus);
             });
diff --git a/src/botPage/view/View.js b/src/botPage/view/View.js
@@ -534,7 +534,6 @@ export default class View {
                 elStopButton.style.display = 'inline-block';
             });
 
-            globalObserver.emit('summary.disable_clear');
             showSummary();
             this.blockly.run(limitations);
         };

Original file line number	Diff line number	Diff line change
`@@ -11,6 +11,7 @@ export default class ClearButton extends React.PureComponent {`
`11`	`11`	`componentDidMount() {`
`12`	`12`	`globalObserver.register('summary.enable_clear', () => this.setState({ isButtonDisabled: false }));`
`13`	`13`	`globalObserver.register('summary.disable_clear', () => this.setState({ isButtonDisabled: true }));`
	`14`	`+ globalObserver.register('bot.running', () => this.setState({ isButtonDisabled: true }));`
`14`	`15`	`}`
`15`	`16`	`// eslint-disable-next-line class-methods-use-this`
`16`	`17`	`confirmClearLog() {`