From 721e26d8e4e45594e636ace473ae58ee9e60213a Mon Sep 17 00:00:00 2001
From: Alexis <35.alexis.pele@gmail.com>
Date: Sat, 8 Jun 2024 21:53:16 +0200
Subject: [PATCH] Added loading and cancelling to updating popover

---
 assets/main.css                          | 14 ++++--
 components/calendar/form/CreateEvent.vue | 11 +++--
 components/calendar/form/UpdateEvent.vue | 62 +++++++++++++++++++-----
 3 files changed, 67 insertions(+), 20 deletions(-)

diff --git a/assets/main.css b/assets/main.css
index 219337f..ad39aa5 100644
--- a/assets/main.css
+++ b/assets/main.css
@@ -75,13 +75,19 @@
   }
 }
 
-.fade-cancel-enter-active,
-.fade-cancel-leave-active {
+.fade-enter-active,
+.fade-leave-active {
+  transition: all .5s ease;
+}
+.fade-delay-enter-active,
+.fade-delay-leave-active {
   transition: all .5s ease 1s;
 }
 
-.fade-cancel-enter-from,
-.fade-cancel-leave-to {
+.fade-enter-from,
+.fade-leave-to,
+.fade-delay-enter-from,
+.fade-delay-leave-to {
   opacity: 0;
   visibility: hidden;
 }
diff --git a/components/calendar/form/CreateEvent.vue b/components/calendar/form/CreateEvent.vue
index ac6cec8..524121f 100644
--- a/components/calendar/form/CreateEvent.vue
+++ b/components/calendar/form/CreateEvent.vue
@@ -21,14 +21,12 @@ const props = defineProps<{
  * Opens event creation's popover
  */
 function openEventCreatePopover() {
-  console.log(operationInProgress.value)
+  // If another operation is in progress, whether it's another create popup or a modal, don't bother opening it
   if (operationInProgress.value) {
     popoverOpen.value = false
     return
   }
 
-  console.log('open still what ?')
-
   resetSkeleton()
 
   popoverOpen.value = true
@@ -59,6 +57,7 @@ async function handleSubmit() {
 }
 
 /**
+ * Prevents the modal from closing if's still loading
  *
  * @param e The closing event (can be keydown or click)
  */
@@ -70,6 +69,8 @@ function handleClosing(e: Event) {
 
 /**
  * Click on the cancel button
+ *
+ * Must cancel the abortController in the store, and stop the loading
  */
 function handleCancel() {
   cancelLatestRequest()
@@ -159,14 +160,14 @@ function handleCancel() {
           </div>
 
           <div class="flex gap-2 justify-end">
-            <Transition name="fade-cancel">
+            <Transition name="fade-delay">
               <UiButton v-if="isLoading" type="button" size="sm" variant="destructive" @click.prevent="handleCancel">
                 Annuler
               </UiButton>
             </Transition>
 
             <UiButton size="sm" :disabled="isLoading">
-              <Transition name="fade-cancel">
+              <Transition name="fade">
                 <PhCircleNotch v-if="isLoading" size="20" class="opacity-50 animate-spin"/>
               </Transition>
 
diff --git a/components/calendar/form/UpdateEvent.vue b/components/calendar/form/UpdateEvent.vue
index 7350394..040a2c8 100644
--- a/components/calendar/form/UpdateEvent.vue
+++ b/components/calendar/form/UpdateEvent.vue
@@ -1,12 +1,14 @@
 <script lang="ts" setup>
-import { PhAlarm, PhMapPinArea } from '@phosphor-icons/vue'
+import { PhAlarm, PhCircleNotch, PhMapPinArea } from '@phosphor-icons/vue'
 import { VisuallyHidden } from 'radix-vue'
 
 const { isEditEventModalOpen } = storeToRefs(useCalendarEvents())
 
-const { resetSkeleton, updateEventFromSkeleton } = useCalendarEvents()
+const { resetSkeleton, updateEventFromSkeleton, cancelLatestRequest } = useCalendarEvents()
 const { eventSkeleton, lastActiveEvent } = storeToRefs(useCalendarEvents())
 
+const isLoading = ref(false)
+
 const formErrors = reactive<{ message: string | null }>({
   message: null
 })
@@ -19,6 +21,10 @@ watch(isEditEventModalOpen, (hasOpened, _o) => {
 })
 
 async function handleAction() {
+  if (isLoading.value) return
+
+  isLoading.value = true
+
   try {
     await updateEventFromSkeleton()
 
@@ -29,8 +35,30 @@ async function handleAction() {
     }
   } finally {
     resetSkeleton()
+    isLoading.value = false
   }
 }
+
+/**
+ * Prevents the modal from closing if's still loading
+ *
+ * @param e The closing event (can be keydown or click)
+ */
+function handleClosing(e: Event) {
+  if (isLoading.value) {
+    e.preventDefault()
+  }
+}
+
+/**
+ * Click on the cancel button
+ *
+ * Must cancel the abortController in the store, and stop the loading
+ */
+function handleCancel() {
+  cancelLatestRequest()
+  isLoading.value = false
+}
 </script>
 
 <template>
@@ -42,6 +70,10 @@ async function handleAction() {
       :disable-outside-pointer-events="true"
       :trap-focus="true"
       class="pl-3 min-w-96 bg-slate-900 border-slate-800"
+      @escape-key-down="handleClosing"
+      @focus-outside="handleClosing"
+      @interact-outside="handleClosing"
+      @pointer-down-outside="handleClosing"
     >
       <VisuallyHidden>
         <UiAlertDialogTitle> Modifier l'évènement</UiAlertDialogTitle>
@@ -51,7 +83,7 @@ async function handleAction() {
         </UiAlertDialogDescription>
       </VisuallyHidden>
 
-      <form>
+      <form @submit.prevent="handleAction">
         <div class="grid grid-cols-2 gap-y-4">
           <div class="col-span-2 ml-8">
             <input
@@ -115,15 +147,23 @@ async function handleAction() {
             </span>
           </div>
         </div>
+
+        <div class="flex gap-2 justify-end">
+          <Transition name="fade-delay">
+            <UiButton v-if="isLoading" type="button" size="sm" variant="destructive" @click.prevent="handleCancel">
+              Annuler
+            </UiButton>
+          </Transition>
+
+          <UiButton size="sm" :disabled="isLoading">
+            <Transition name="fade">
+              <PhCircleNotch v-if="isLoading" size="20" class="animate-spin"/>
+            </Transition>
+
+            Sauvegarder
+          </UiButton>
+        </div>
       </form>
-      <UiAlertDialogFooter>
-        <UiAlertDialogCancel>
-          Annuler
-        </UiAlertDialogCancel>
-        <UiAlertDialogAction @click="handleAction">
-          Sauvegarder
-        </UiAlertDialogAction>
-      </UiAlertDialogFooter>
     </UiAlertDialogContent>
   </UiAlertDialog>
 </template>