From 313da56d92ae1f68f9c1feba44600a93a81bff94 Mon Sep 17 00:00:00 2001
From: Tixie <admin@glitch.family>
Date: Tue, 30 Apr 2019 19:19:26 +0200
Subject: [PATCH] refactor(preview): Move canvas in a dedicated component

---
 src/views/components/preview-canvas.vue | 71 +++++++++++++++++++++++++
 src/views/screens/preview.vue           | 55 ++-----------------
 2 files changed, 75 insertions(+), 51 deletions(-)
 create mode 100644 src/views/components/preview-canvas.vue

diff --git a/src/views/components/preview-canvas.vue b/src/views/components/preview-canvas.vue
new file mode 100644
index 0000000..bc6f1e5
--- /dev/null
+++ b/src/views/components/preview-canvas.vue
@@ -0,0 +1,71 @@
+<template>
+  <canvas ref="previewCanvas"></canvas>
+</template>
+
+<script>
+import { mapState } from 'vuex'
+import { boomerang } from '/services/effects.js'
+import { pipe, cycle } from '/services/util.js'
+
+export default {
+  name: 'previewCanvas',
+  data: () => ({
+    previewInterval: null
+  }),
+  computed: {
+    ...mapState([
+      'capture',
+      'boomerang'
+    ])
+  },
+  methods: {
+    startPreview () {
+      if (!this.capture) {
+        this.backHome()
+      } else {
+        const canvas = this.$refs.previewCanvas
+        canvas.width = this.capture.imageWidth
+        canvas.height = this.capture.imageHeight
+        const canvasContext = canvas.getContext('2d')
+
+        const imagesIterator = pipe(this.capture.imageDataList, [
+          (images) => this.boomerang ? boomerang(images) : images,
+          cycle
+        ])
+        const delay = this.capture.delayTime
+
+        this.previewInterval = setInterval(() => {
+          const {
+            value: image,
+            done
+          } = imagesIterator.next()
+
+          if (image) {
+            canvasContext.putImageData(image, 0, 0)
+          }
+
+          if (done) {
+            clearInterval(this.previewInterval)
+            this.previewInterval = null
+          }
+        }, delay)
+      }
+    },
+    stopPreview () {
+      window.clearInterval(this.previewInterval)
+    }
+  },
+  watch: {
+    boomerang: function () {
+      this.stopPreview()
+      this.startPreview()
+    }
+  },
+  mounted () {
+    this.startPreview()
+  },
+  destroyed () {
+    this.stopPreview()
+  }
+}
+</script>
diff --git a/src/views/screens/preview.vue b/src/views/screens/preview.vue
index 32a75f5..5c3af6f 100644
--- a/src/views/screens/preview.vue
+++ b/src/views/screens/preview.vue
@@ -3,7 +3,7 @@
     <capture-options :disabled-time="true" :disabled-timer="true" :back-btn="back"></capture-options>
 
     <div class="preview">
-      <canvas ref="previewCanvas" class="preview-visual"></canvas>
+      <preview-canvas v-if="capture" class="preview-visual"></preview-canvas>
     </div>
 
     <button class="download-btn btn btn--primary w100" :class="{ 'btn--loading': encoding }" :disabled="encoding" @click="startEncoding">Generate GIF</button>
@@ -16,8 +16,7 @@
 import { encode } from '/services/encode.js'
 import encodingOverlay from '/views/components/encoding'
 import captureOptions from '/views/components/capture-options'
-import { boomerang } from '/services/effects.js'
-import { pipe, cycle } from '/services/util.js'
+import previewCanvas from '/views/components/preview-canvas'
 
 import { mapState } from 'vuex'
 
@@ -25,7 +24,8 @@ export default {
   name: 'preview',
   components: {
     encodingOverlay,
-    captureOptions
+    captureOptions,
+    previewCanvas
   },
   data: () => ({
     encoding: false,
@@ -46,41 +46,6 @@ export default {
     backHome () {
       this.$router.push({ name: 'home' })
     },
-    startPreview () {
-      if (!this.capture) {
-        this.backHome()
-      } else {
-        const canvas = this.$refs.previewCanvas
-        canvas.width = this.capture.imageWidth
-        canvas.height = this.capture.imageHeight
-        const canvasContext = canvas.getContext('2d')
-
-        const imagesIterator = pipe(this.capture.imageDataList, [
-          (images) => this.boomerang ? boomerang(images) : images,
-          cycle
-        ])
-        const delay = this.capture.delayTime
-
-        this.previewInterval = setInterval(() => {
-          const {
-            value: image,
-            done
-          } = imagesIterator.next()
-
-          if (image) {
-            canvasContext.putImageData(image, 0, 0)
-          }
-
-          if (done) {
-            clearInterval(this.previewInterval)
-            this.previewInterval = null
-          }
-        }, delay)
-      }
-    },
-    stopPreview () {
-      window.clearInterval(this.previewInterval)
-    },
     startEncoding () {
       this.encoding = true
       const encoding = encode(this.capture, { boomerangEffect: this.boomerang })
@@ -103,22 +68,10 @@ export default {
       })
     }
   },
-  watch: {
-    boomerang: function () {
-      this.stopPreview()
-      this.startPreview()
-    }
-  },
   created () {
     if (!this.capture) {
       this.backHome()
     }
-  },
-  mounted () {
-    this.startPreview()
-  },
-  destroyed () {
-    this.stopPreview()
   }
 }
 </script>