From 2b70ec8c8d6b1815f38367afa68c794a5e835330 Mon Sep 17 00:00:00 2001
From: Arturo Manzoli <arturomanzoli@gmail.com>
Date: Tue, 7 Jan 2025 17:57:58 -0300
Subject: [PATCH] App: Startup dialogs: Implement changes to run under dialog
 queue

Signed-off-by: Arturo Manzoli <arturomanzoli@gmail.com>
---
 src/components/Tutorial.vue               | 21 ++++++---------------
 src/components/UpdateNotification.vue     | 17 ++++++++++++++---
 src/components/VehicleDiscoveryDialog.vue | 23 +++++++++++++++++------
 3 files changed, 37 insertions(+), 24 deletions(-)

diff --git a/src/components/Tutorial.vue b/src/components/Tutorial.vue
index edf4d61a7..78c7878d1 100644
--- a/src/components/Tutorial.vue
+++ b/src/components/Tutorial.vue
@@ -88,23 +88,15 @@ import { useMainVehicleStore } from '@/stores/mainVehicle'
 import GlassModal from './GlassModal.vue'
 
 const { showSnackbar } = useSnackbar()
+
 const interfaceStore = useAppInterfaceStore()
 const vehicleStore = useMainVehicleStore()
 
-const props = defineProps<{
-  /**
-   *
-   */
-  showTutorial?: boolean
-}>()
-
-const emits = defineEmits(['update:showTutorial'])
-
-const showTutorial = ref(props.showTutorial || false)
+const userHasSeenTutorial = useBlueOsStorage('cockpit-has-seen-tutorial', false)
+const showTutorial = ref(false)
 const currentTutorialStep = ref(1)
 const isVehicleConnectedVisible = ref(false)
 const tallContent = ref(false)
-const userHasSeenTutorial = useBlueOsStorage('cockpit-has-seen-tutorial', false)
 
 const steps = [
   {
@@ -345,7 +337,6 @@ const handleStepChangeDown = (newStep: number): void => {
 const dontShowTutorialAgain = (): void => {
   userHasSeenTutorial.value = true
   showTutorial.value = false
-  emits('update:showTutorial', false)
   showSnackbar({
     message: 'This guide can be reopened via the Settings > General menu',
     variant: 'info',
@@ -356,8 +347,6 @@ const dontShowTutorialAgain = (): void => {
 
 const alwaysShowTutorialOnStartup = (): void => {
   userHasSeenTutorial.value = false
-  showTutorial.value = true
-  emits('update:showTutorial', true)
 }
 
 const nextTutorialStep = (): void => {
@@ -379,7 +368,7 @@ const closeTutorial = (): void => {
   showTutorial.value = false
   interfaceStore.componentToHighlight = 'none'
   currentTutorialStep.value = 1
-  emits('update:showTutorial', false)
+  interfaceStore.isTutorialVisible = false
 }
 
 const setVehicleConnectedVisible = (): void => {
@@ -403,6 +392,7 @@ const handleKeydown = (event: KeyboardEvent): void => {
 watch(
   () => interfaceStore.isTutorialVisible,
   (val) => {
+    console.log('🚀 ~ val:', val)
     showTutorial.value = val
   }
 )
@@ -435,6 +425,7 @@ onMounted(() => {
 
 onBeforeUnmount(() => {
   window.removeEventListener('keydown', handleKeydown)
+  interfaceStore.isTutorialVisible = false
 })
 </script>
 <style>
diff --git a/src/components/UpdateNotification.vue b/src/components/UpdateNotification.vue
index c79521856..b460d918c 100644
--- a/src/components/UpdateNotification.vue
+++ b/src/components/UpdateNotification.vue
@@ -27,6 +27,7 @@
         </template>
       </v-progress-linear>
     </template>
+    <template #actions> <v-btn variant="text" size="small" @click="showUpdateDialog = false">Close</v-btn> </template>
   </InteractionDialog>
 </template>
 
@@ -37,6 +38,9 @@ import { onBeforeMount, ref } from 'vue'
 import InteractionDialog, { type Action } from '@/components/InteractionDialog.vue'
 import { app_version } from '@/libs/cosmos'
 import { isElectron } from '@/libs/utils'
+import { useAppInterfaceStore } from '@/stores/appInterface'
+
+const interfaceStore = useAppInterfaceStore()
 
 const showUpdateDialog = ref(false)
 const dialogTitle = ref('')
@@ -75,7 +79,10 @@ onBeforeMount(() => {
     dialogVariant.value = 'info'
     dialogActions.value = []
     showProgress.value = false
-    showUpdateDialog.value = true
+    if (interfaceStore.activeDialog?.id === 'UpdateNotification' || interfaceStore.activeDialog === undefined) {
+      showUpdateDialog.value = true
+      return
+    }
   })
 
   window.electronAPI.onUpdateNotAvailable(() => {
@@ -144,7 +151,9 @@ onBeforeMount(() => {
       return
     }
 
-    showUpdateDialog.value = true
+    if (interfaceStore.activeDialog?.id === 'UpdateNotification' || interfaceStore.activeDialog === undefined) {
+      showUpdateDialog.value = true
+    }
   })
 
   window.electronAPI.onDownloadProgress((progressInfo) => {
@@ -175,7 +184,9 @@ onBeforeMount(() => {
         },
       },
     ]
-    showUpdateDialog.value = true
+    if (interfaceStore.activeDialog?.id === 'UpdateNotification' || interfaceStore.activeDialog === undefined) {
+      showUpdateDialog.value = true
+    }
   })
 })
 </script>
diff --git a/src/components/VehicleDiscoveryDialog.vue b/src/components/VehicleDiscoveryDialog.vue
index 456f5fb99..1f656a147 100644
--- a/src/components/VehicleDiscoveryDialog.vue
+++ b/src/components/VehicleDiscoveryDialog.vue
@@ -47,15 +47,18 @@
 
 <script setup lang="ts">
 import { useStorage } from '@vueuse/core'
-import { ref, watch } from 'vue'
+import { computed, ref, watch } from 'vue'
 
 import { useSnackbar } from '@/composables/snackbar'
 import vehicleDiscover, { NetworkVehicle } from '@/libs/electron/vehicle-discovery'
 import { reloadCockpit } from '@/libs/utils'
+import { useAppInterfaceStore } from '@/stores/appInterface'
 import { useMainVehicleStore } from '@/stores/mainVehicle'
 
 import InteractionDialog, { Action } from './InteractionDialog.vue'
 
+const interfaceStore = useAppInterfaceStore()
+
 const props = defineProps<{
   /**
    *
@@ -69,13 +72,14 @@ const props = defineProps<{
 
 const emit = defineEmits<{
   (e: 'update:modelValue', value: boolean): void
+  (e: 'close'): void
 }>()
 
 const { showSnackbar } = useSnackbar()
 const mainVehicleStore = useMainVehicleStore()
 const discoveryService = vehicleDiscover
 
-const isOpen = ref(props.modelValue)
+const isOpen = ref(false)
 const searching = ref(false)
 const searched = ref(false)
 const vehicles = ref<NetworkVehicle[]>([])
@@ -85,7 +89,7 @@ const originalActions = [
   {
     text: 'Close',
     action: () => {
-      isOpen.value = false
+      handleCloseDialog()
     },
   },
 ]
@@ -105,7 +109,6 @@ watch(
     isOpen.value = value
   }
 )
-
 watch(isOpen, (value) => {
   emit('update:modelValue', value)
 })
@@ -114,6 +117,8 @@ const searchVehicles = async (): Promise<void> => {
   searching.value = true
   disableButtons()
   vehicles.value = await discoveryService.findVehicles()
+
+  handleCloseDialog()
   searching.value = false
   enableButtons()
   searched.value = true
@@ -121,7 +126,8 @@ const searchVehicles = async (): Promise<void> => {
 
 const selectVehicle = async (address: string): Promise<void> => {
   mainVehicleStore.globalAddress = address
-  isOpen.value = false
+  handleCloseDialog()
+
   await reloadCockpit()
   showSnackbar({ message: 'Vehicle address updated', variant: 'success', duration: 5000 })
 }
@@ -130,10 +136,15 @@ const preventFutureAutoSearchs = (): void => {
   preventAutoSearch.value = true
   disableButtons()
   setTimeout(() => {
-    isOpen.value = false
+    handleCloseDialog()
   }, 5000)
 }
 
+const handleCloseDialog = (): void => {
+  isOpen.value = false
+  emit('close')
+}
+
 const disableButtons = (): void => {
   dialogActions.value = originalActions.map((action) => ({ ...action, disabled: true }))
 }