From 6dde17e7d3b73507d3baf613d2dafaf956578863 Mon Sep 17 00:00:00 2001
From: Joel Collins <joel@jtcollins.net>
Date: Tue, 26 Feb 2019 10:47:58 +0000
Subject: [PATCH] Minor tidying

---
 .../paneDisplayComponents/streamDisplay.vue          |  6 ++++--
 .../paneSettingsComponents/appSettings.vue           |  2 +-
 src/components/panelDisplay.vue                      | 12 +-----------
 3 files changed, 6 insertions(+), 14 deletions(-)

diff --git a/src/components/paneDisplayComponents/streamDisplay.vue b/src/components/paneDisplayComponents/streamDisplay.vue
index fdd0bdf5..906845c2 100644
--- a/src/components/paneDisplayComponents/streamDisplay.vue
+++ b/src/components/paneDisplayComponents/streamDisplay.vue
@@ -42,10 +42,12 @@ export default {
 	},
 
 	created: function () {
+		// Add resize listener
   	window.addEventListener('resize', this.handleResize)
 	},
 
 	beforeDestroy: function () {
+		// Remove resize listener
 		window.removeEventListener('resize', this.handleResize)
 	},
 
@@ -66,6 +68,7 @@ export default {
 		},
 		
 		handleResize: function(event) {
+			// Only fires resize event after no resize in 500ms (prevents resize event spam)
 			clearTimeout(this.resizeTeimoutId);
 			this.resizeTeimoutId = setTimeout(this.handleDoneResize, 500)
 		},
@@ -101,8 +104,7 @@ export default {
 
     previewRequest: function(state) {
 			if (this.$store.getters.ready == true) {
-				// Create URI depending on if starting or stopping preview
-				// TODO: Messy. Should be cleaned up
+				// Create URI and set this.GpuPreviewActive depending on if starting or stopping preview
 				if (state == true) {
 					this.GpuPreviewActive = true
 					var requestUri = this.startPreviewUri;
diff --git a/src/components/paneSettingsComponents/appSettings.vue b/src/components/paneSettingsComponents/appSettings.vue
index 935b01bb..e6a3cf84 100644
--- a/src/components/paneSettingsComponents/appSettings.vue
+++ b/src/components/paneSettingsComponents/appSettings.vue
@@ -4,7 +4,7 @@
   <p><label><input v-model="disableStream" class="uk-checkbox" type="checkbox"> Disable stream</label></p>
 
   <div class="uk-child-width-1-2" uk-grid>
-    <p><label><input v-model="autoGpuPreview" class="uk-checkbox" type="checkbox"> GPU preview</label></p>
+    <p><label v-bind:class="{'uk-disabled': !this.$store.getters.ready}"><input v-model="autoGpuPreview" class="uk-checkbox" type="checkbox"> GPU preview</label></p>
     <p><label><input v-model="trackWindow" class="uk-checkbox" type="checkbox"> Track window</label></p>
   </div>
 
diff --git a/src/components/panelDisplay.vue b/src/components/panelDisplay.vue
index 6a498e2d..aa5b511f 100644
--- a/src/components/panelDisplay.vue
+++ b/src/components/panelDisplay.vue
@@ -2,7 +2,7 @@
   <div id="panelDisplay" class="uk-flex uk-flex-column uk-margin-remove uk-padding-remove uk-height-1-1">
     <ul class="uk-flex-none uk-flex-center uk-margin-remove-bottom uk-text-center" uk-tab="swiping: false">
       <li><a href="#" uk-switcher-item="preview" uk-icon="play-circle" uk-tooltip="Live"></a></li>
-      <li v-bind:class="disableIfDisconnected"><a href="#" uk-switcher-item="gallery" uk-icon="image" uk-tooltip="Captures"></a></li>
+      <li v-bind:class="{'uk-disabled': !this.$store.getters.ready}"><a href="#" uk-switcher-item="gallery" uk-icon="image" uk-tooltip="Captures"></a></li>
     </ul>
     <ul class="uk-switcher uk-flex uk-flex-1">
       <li class="uk-height-1-1 uk-width-1-1 clickableTab" id="streamDisplayTab"><streamDisplay/></li>
@@ -48,20 +48,10 @@ export default {
       }
     });
 
-  },
-
-
-  computed: {
-    disableIfDisconnected: function () {
-      return {
-        'uk-disabled': !this.$store.getters.ready
-      }
-    }
   }
 
 }
 </script>
 
 <style scoped lang="less">
-
 </style>