Remove some JobComponent state attributes.

Author: vlad-shatskyi

src/views/3_JobComponent.tsx

@@ -11,8 +11,6 @@ interface Props {
 }
 
 interface State {
-    status?: e.Status;
-    canBeDecorated?: boolean;
     decorate?: boolean;
 }
 
@@ -21,9 +19,7 @@ export default class JobComponent extends React.Component<Props, State> implemen
         super(props);
 
         this.state = {
-            status: this.props.job.status,
             decorate: false,
-            canBeDecorated: false,
         };
 
         // FIXME: find a better design to propagate events.
@@ -34,8 +30,8 @@ export default class JobComponent extends React.Component<Props, State> implemen
 
     componentDidMount() {
         this.props.job
-            .on("data", () => this.setState({canBeDecorated: this.props.job.canBeDecorated()}))
-            .on("status", (status: e.Status) => this.setState({status: status}));
+            .on("data", () => this.forceUpdate())
+            .on("status", () => this.forceUpdate())
     }
 
     componentDidUpdate() {
@@ -47,16 +43,16 @@ export default class JobComponent extends React.Component<Props, State> implemen
 
     render() {
         let buffer: React.ReactElement<any>;
-        if (this.state.canBeDecorated && this.state.decorate) {
+        if (this.props.job.canBeDecorated() && this.state.decorate) {
             buffer = this.props.job.decorate();
         } else {
             buffer = <BufferComponent job={this.props.job}/>;
         }
 
         return (
-            <div className={"job " + this.state.status}>
+            <div className={"job"}>
                 <PromptComponent job={this.props.job}
-                                 status={this.state.status}
+                                 status={this.props.job.status}
                                  hasLocusOfAttention={this.props.hasLocusOfAttention}
                                  jobView={this}/>
                 {buffer}
@@ -70,7 +66,7 @@ export default class JobComponent extends React.Component<Props, State> implemen
             return;
         }
 
-        if (this.state.status === e.Status.InProgress && !event.metaKey && !isModifierKey(event)) {
+        if (this.props.job.status === e.Status.InProgress && !event.metaKey && !isModifierKey(event)) {
             if (keys.interrupt(event)) {
                 this.props.job.interrupt();
             } else {

src/views/4_PromptComponent.tsx

@@ -162,7 +162,7 @@ export default class PromptComponent extends React.Component<Props, State> imple
             }
         }
 
-        if (this.props.jobView.state.canBeDecorated) {
+        if (this.props.job.canBeDecorated()) {
             decorationToggle = <DecorationToggleComponent job={this.props.jobView}/>;
         }
 

tsconfig.json

@@ -8,7 +8,6 @@
     "experimentalDecorators": true,
     "noImplicitAny": true,
     "noEmitOnError": true,
-    "pretty": true,
     "jsx": "react",
     "outDir": "compiled/src"
   },