From b19113f232ce54eccc77751ad7c3d72e6eab9110 Mon Sep 17 00:00:00 2001
From: Alexis <35.alexis.pele@gmail.com>
Date: Tue, 9 Jan 2024 17:40:55 +0100
Subject: [PATCH] Added disabled button style

---
 src/assets/scss/_buttons.scss                 |  9 +++++++
 src/components/maps/overlay/MarkerCreator.vue | 27 ++++++++++++++-----
 2 files changed, 30 insertions(+), 6 deletions(-)

diff --git a/src/assets/scss/_buttons.scss b/src/assets/scss/_buttons.scss
index 9f02b46f..c6170fd5 100644
--- a/src/assets/scss/_buttons.scss
+++ b/src/assets/scss/_buttons.scss
@@ -13,6 +13,10 @@
     transition-duration: .2s;
     transition-timing-function: cubic-bezier(0.785, 0.135, 0.15, 0.86);
 
+    &:disabled {
+        cursor: not-allowed;
+    }
+
     &-secondary {
         &:not(:disabled) {
             background-color: var(--slate-200);
@@ -23,6 +27,11 @@
                 background-color: var(--blue-500);
             }
         }
+
+        &:disabled {
+            color: var(--slate-500);
+            background-color: var(--slate-100);
+        }
     }
 
     &-danger {
diff --git a/src/components/maps/overlay/MarkerCreator.vue b/src/components/maps/overlay/MarkerCreator.vue
index 631fcea8..febbcd18 100644
--- a/src/components/maps/overlay/MarkerCreator.vue
+++ b/src/components/maps/overlay/MarkerCreator.vue
@@ -1,17 +1,17 @@
 <script lang="ts" setup>
-import { onClickOutside, onLongPress, useCssVar, useFocus, useLocalStorage, useMouse, useTimeout, useTimeoutFn } from '@vueuse/core';
-import { computed, onMounted, onUnmounted, ref, watch } from 'vue';
+import { onClickOutside, onLongPress, useCssVar, useFocus, useLocalStorage, useMouse, useTimeout, useTimeoutFn } from '@vueuse/core'
+import { computed, onMounted, onUnmounted, ref, watch } from 'vue'
 
-const markerMenu = ref<HTMLMenuElement>();
+const markerMenu = ref<HTMLMenuElement>()
 
-type MenuMode = "editing-marker" | "default";
+type MenuMode = "editing-marker" | "default"
 
 /**
  * Appearing behaviour
  */
 const isActive = ref<Boolean>(false)
 const shouldBeShown = computed(() => isActive.value)
-const currentMode = ref<MenuMode>("default");
+const currentMode = ref<MenuMode>("default")
 
 function switchMenuMode(newMode: MenuMode) {
     currentMode.value = newMode
@@ -76,7 +76,6 @@ watch( isActive, (o, n) => {
 function updateMenuPosition() {
     updateCSSVars()
 }
-
 function updateCSSVars() {
     mouseXVar.value = `${mouse.x.value.toFixed()}px`
     mouseYVar.value = `${mouse.y.value.toFixed()}px`
@@ -105,7 +104,12 @@ watch(currentMode, (n, o) => {
 // Closes the menu if clicked outside
 onClickOutside(markerMenu, handleClickOutside, { ignore: [markerModal] })
 
+/**
+ * Add Custom Marker
+ */
 function handleAddCustomMarker() {
+    if (!markerTitle.value) return
+
     const addCustomMarkerEvent = new CustomEvent(`add-custom-marker`, { bubbles: true, detail: { title: markerTitle.value }})
 
     addCustomMarker.value?.dispatchEvent(addCustomMarkerEvent)
@@ -116,18 +120,28 @@ function handleAddCustomMarker() {
     markerTitle.value = ""
 }
 
+/**
+ * Hide modal on cancel and resets 
+ */
 function handleMarkerModalCancel() {
     showMenu()
     hideMarkerModal()
     switchMenuMode('default')
 }
 
+/**
+ * Display marker modal
+ */
 function showMarkerModal() {
     markerModal.value?.showModal()
     markerModalOpen.value = true
     switchMenuMode('editing-marker')
 }
 
+/**
+ * Hide marker modal
+ * TODO: Implement a timer for animation
+ */
 function hideMarkerModal() {
     markerModal.value?.close()
     markerModalOpen.value = false
@@ -177,6 +191,7 @@ function hideMarkerModal() {
                         ref="addCustomMarker"
                         type="submit"
                         class="btn btn-secondary"
+                        :disabled="!markerTitle"
                     >
                         <span>Créer</span>
                     </button>