From c3786587ffd6a7d4f44815f5b22b202f18dbe2b7 Mon Sep 17 00:00:00 2001
From: Guillaume Chau <alphadelta.fg@gmail.com>
Date: Fri, 15 Jun 2018 15:44:48 +0200
Subject: [PATCH] fix(ui): improved performance

---
 .../src/components/WebpackAnalyzer.vue        |  2 ++
 .../src/components/ConnectionStatus.vue       |  6 +++++
 .../@vue/cli-ui/src/components/StatusBar.vue  |  2 ++
 .../cli-ui/src/components/TerminalView.vue    | 27 +++++++++----------
 .../@vue/cli-ui/src/mixins/ClientState.js     | 23 +++++++++++++---
 5 files changed, 41 insertions(+), 19 deletions(-)

diff --git a/packages/@vue/cli-ui-addon-webpack/src/components/WebpackAnalyzer.vue b/packages/@vue/cli-ui-addon-webpack/src/components/WebpackAnalyzer.vue
index cee0087c9e..037a57ee1e 100644
--- a/packages/@vue/cli-ui-addon-webpack/src/components/WebpackAnalyzer.vue
+++ b/packages/@vue/cli-ui-addon-webpack/src/components/WebpackAnalyzer.vue
@@ -118,6 +118,8 @@ import { size } from '../filters'
 import DonutModule from './DonutModule.vue'
 
 export default {
+  clientState: true,
+
   mixins: [
     Dashboard
   ],
diff --git a/packages/@vue/cli-ui/src/components/ConnectionStatus.vue b/packages/@vue/cli-ui/src/components/ConnectionStatus.vue
index 675cffef01..b57ad6d065 100644
--- a/packages/@vue/cli-ui/src/components/ConnectionStatus.vue
+++ b/packages/@vue/cli-ui/src/components/ConnectionStatus.vue
@@ -24,6 +24,12 @@
   </ApolloQuery>
 </template>
 
+<script>
+export default {
+  clientState: true
+}
+</script>
+
 <style lang="stylus" scoped>
 @import "~@/style/imports"
 
diff --git a/packages/@vue/cli-ui/src/components/StatusBar.vue b/packages/@vue/cli-ui/src/components/StatusBar.vue
index 3c2fcc4937..933ef0f701 100644
--- a/packages/@vue/cli-ui/src/components/StatusBar.vue
+++ b/packages/@vue/cli-ui/src/components/StatusBar.vue
@@ -83,6 +83,8 @@ import DARK_MODE_SET from '../graphql/darkModeSet.gql'
 let lastRoute
 
 export default {
+  clientState: true,
+
   data () {
     return {
       showLogs: false,
diff --git a/packages/@vue/cli-ui/src/components/TerminalView.vue b/packages/@vue/cli-ui/src/components/TerminalView.vue
index 579ee28c5c..d34ebb790c 100644
--- a/packages/@vue/cli-ui/src/components/TerminalView.vue
+++ b/packages/@vue/cli-ui/src/components/TerminalView.vue
@@ -73,6 +73,8 @@ const darkTheme = {
 }
 
 export default {
+  clientState: true,
+
   props: {
     cols: {
       type: Number,
@@ -136,21 +138,12 @@ export default {
 
     content: 'setContent',
 
-    theme: {
-      handler (value) {
-        if (this.$_terminal) {
-          this.$_terminal._setTheme(this.theme)
-        }
-      },
-      immediate: true
-    }
-  },
-
-  mounted () {
-    this.initTerminal()
-
-    if (this.autoSize) {
-      this.$nextTick(this.fit)
+    darkMode (value, oldValue) {
+      if (typeof oldValue === 'undefined') {
+        this.initTerminal()
+      } else if (this.$_terminal) {
+        this.$_terminal._setTheme(this.theme)
+      }
     }
   },
 
@@ -171,6 +164,10 @@ export default {
 
       term.on('blur', () => this.$emit('blur'))
       term.on('focus', () => this.$emit('focus'))
+
+      if (this.autoSize) {
+        this.$nextTick(this.fit)
+      }
     },
 
     setContent (value, ln = true) {
diff --git a/packages/@vue/cli-ui/src/mixins/ClientState.js b/packages/@vue/cli-ui/src/mixins/ClientState.js
index 45042a9980..3bc0bd1437 100644
--- a/packages/@vue/cli-ui/src/mixins/ClientState.js
+++ b/packages/@vue/cli-ui/src/mixins/ClientState.js
@@ -3,9 +3,24 @@ import DARK_MODE from '../graphql/darkMode.gql'
 
 // @vue/component
 export default {
-  apollo: {
-    // In plugins, user `this.$root.connected` for example
-    connected: CONNECTED,
-    darkMode: DARK_MODE
+  beforeCreate () {
+    if (this.$options.clientState) {
+      const newData = {
+        connected: CONNECTED,
+        darkMode: DARK_MODE
+      }
+      this.$options.apollo = {
+        ...this.$options.apollo,
+        ...newData
+      }
+      // Proxy prop on `this`
+      for (const key in newData) {
+        Object.defineProperty(this, key, {
+          get: () => this.$data.$apolloData.data[key],
+          enumerable: true,
+          configurable: true
+        })
+      }
+    }
   }
 }