Clean up sidebar shift flow

Ives van Hoorne · Ives van Hoorne · commit fb844e0c193c · 2017-12-08T16:52:36.000+01:00
diff --git a/packages/app/src/app/pages/Sandbox/Editor/index.js b/packages/app/src/app/pages/Sandbox/Editor/index.js
@@ -35,9 +35,6 @@ export default class ContentSplit extends React.PureComponent<Props, State> {
     if (e.keyCode === 16) {
       if (!this.shiftPressed) {
         this.shiftPressed = true;
-        if (this.timeout) {
-          clearTimeout(this.timeout);
-        }
         this.timeout = setTimeout(() => {
           this.shiftPressed = false;
         }, 500);
@@ -53,6 +50,9 @@ export default class ContentSplit extends React.PureComponent<Props, State> {
 
   componentWillUnmount() {
     document.removeEventListener('keydown', this.handleKeyPress);
+    if (this.timeout) {
+      clearTimeout(this.timeout);
+    }
   }
 
   toggleWorkspace = () =>
