feat(client/App): save workspace on component update

Related to #866 Related to #910
camunda · Jan 17, 2019 · bc10807 · bc10807
1 parent 80362cd
commit bc10807
Showing 1 changed file with 4 additions and 10 deletions.
diff --git a/client/src/app/App.js b/client/src/app/App.js
@@ -199,8 +199,6 @@ export class App extends Component {
     }
 
     await this._removeTab(tab);
-
-    this.saveWorkspace();
   }
 
   isDirty = (tab) => {
@@ -332,14 +330,12 @@ export class App extends Component {
 
     this.setState({
       layout: merge(layout, newLayout)
-    }, () => {
-
-      // wait for new state
-      this.saveWorkspace();
     });
   }
 
   saveWorkspace = () => {
+    console.log('saveWorkspace');
+
     const {
       workspace
     } = this.props.globals;
@@ -416,8 +412,6 @@ export class App extends Component {
         [activeTab.id]: true
       },
       tabLoadingState: 'shown'
-    }, () => {
-      this.saveWorkspace();
     });
   }
 
@@ -473,8 +467,6 @@ export class App extends Component {
         ...dirtyTabs,
         [tab.id]: false
       }
-    }, () => {
-      this.saveWorkspace();
     });
   }
 
@@ -568,6 +560,8 @@ export class App extends Component {
 
     }
 
+    // TODO(philippfromme): only save workspace if necessary
+    this.saveWorkspace();
   }
 
   async saveTab(tab, options = {}) {