feat: better task item name input

resources/ts/components/TaskItem.vue

@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { onBeforeMount, ref, watch, computed, defineAsyncComponent } from "vue";
+import { onBeforeMount, ref, watch, computed, defineAsyncComponent, nextTick } from "vue";
 import type { Task } from "@/types";
 import { CopyIcon, TrashIcon, SaveIcon, XIcon } from "@lucide/vue";
 import { updateTask } from "@/api/tasks";
@@ -11,7 +11,8 @@ import { relativeTime } from "@/lib/dates";
 import { useAlertStore } from "@/stores/alerts";
 import { useI18n } from "vue-i18n";
 
-const emit = defineEmits<(e: "clone" | "delete" | "updated", task: Task) => void>();
+const emit =
+    defineEmits<(e: "clone" | "delete" | "updated", task: Task) => void>();
 
 const task = defineModel<Task>({ required: true });
 defineProps<{ showLastChecked: boolean; showNextReset: boolean }>();
@@ -20,6 +21,7 @@ const editMode = ref<boolean>(false);
 const editName = ref<string>("");
 const editCron = ref<string | null>(null);
 const refreshKey = ref<number>(0);
+const editNameInput = ref<HTMLInputElement | null>(null);
 
 const checked = computed<boolean>(() => taskCompleted(task.value));
 const nextReset = computed<Date | null>(() => taskNextReset(task.value));
@@ -28,13 +30,22 @@ const i18n = useI18n();
 
 const { alertSuccess, alertError } = useAlertStore();
 
+async function onOpen() {
+    editMode.value = true;
+    await nextTick();
+    editNameInput.value?.focus();
+}
+
 function onClose() {
+    editName.value = task.value.name;
+    editCron.value = task.value.reset_cron;
     editMode.value = false;
 }
 
 function onSave() {
+    if (editName.value.trim()) {
         updateTask(task.value.id, {
-        name: editName.value,
+            name: editName.value.trim(),
             reset_cron: editCron.value,
         })
             .then((newTask) => {
@@ -49,6 +60,7 @@ function onSave() {
         task.value.reset_cron = editCron.value;
         editMode.value = false;
     }
+}
 
 function onCheck() {
     if (!checked.value) {
@@ -117,7 +129,7 @@ watch(task, () => {
             </div>
             <div
                 class="cursor-pointer text-lg select-none"
-                @click.prevent="editMode = true"
+                @click.prevent="onOpen"
             >
                 <div>{{ task.name }}</div>
                 <div
@@ -131,7 +143,8 @@ watch(task, () => {
                     v-if="showNextReset && checked && nextReset"
                     class="font-light text-xs italic"
                 >
-                    {{ $t("resets") }} {{ relativeTime(nextReset, $i18n.locale) }}
+                    {{ $t("resets") }}
+                    {{ relativeTime(nextReset, $i18n.locale) }}
                 </div>
             </div>
         </template>
@@ -141,7 +154,13 @@ watch(task, () => {
                     <button class="btn btn-square me-1" @click="onClose">
                         <x-icon />
                     </button>
-                    <button class="btn btn-square me-1" @click="onSave">
+                    <button
+                        class="btn btn-square me-1"
+                        :class="
+                            editName.trim().length === 0 ? 'btn-disabled' : ''
+                        "
+                        @click="onSave"
+                    >
                         <save-icon />
                     </button>
                     <button class="btn btn-square me-1" @click="onClone">
@@ -151,7 +170,13 @@ watch(task, () => {
                         <trash-icon />
                     </button>
                 </div>
-                <input v-model="editName" type="text" class="input w-full" />
+                <input
+                    ref="editNameInput"
+                    v-model="editName"
+                    type="text"
+                    class="input w-full"
+                    @keyup.enter="onSave"
+                />
                 <cron-input v-model="editCron" :edit-mode="editMode" />
             </div>
         </template>