From d6ac93c1d1867b2f1fc06935d74bb88010175a64 Mon Sep 17 00:00:00 2001
From: Paul Elliott <paul.elliott@kitware.com>
Date: Sun, 31 Dec 2023 09:42:45 -0500
Subject: [PATCH] wip

---
 src/components/LabelControls.vue   |  2 +-
 src/components/LabelEditor.vue     | 11 +++++++----
 src/components/ToolLabelEditor.vue | 19 ++++++++++++++++---
 3 files changed, 24 insertions(+), 8 deletions(-)

diff --git a/src/components/LabelControls.vue b/src/components/LabelControls.vue
index c5e671ee2..d2c5cecfc 100644
--- a/src/components/LabelControls.vue
+++ b/src/components/LabelControls.vue
@@ -120,7 +120,7 @@ function deleteEditingLabel() {
   </v-card>
 
   <isolated-dialog v-model="editDialog" max-width="800px">
-    <ToolLabelEditor
+    <tool-label-editor
       v-if="editingLabelID"
       v-model:name="editState.labelName"
       v-model:stroke-width="editState.strokeWidth"
diff --git a/src/components/LabelEditor.vue b/src/components/LabelEditor.vue
index 45ae4f0fe..58d902b11 100644
--- a/src/components/LabelEditor.vue
+++ b/src/components/LabelEditor.vue
@@ -1,11 +1,13 @@
 <script setup lang="ts">
-import { toRefs } from 'vue';
+import { computed, toRefs } from 'vue';
 
 const emit = defineEmits(['done', 'cancel', 'delete', 'update:color']);
-const props = defineProps({
-  color: String,
+const props = defineProps<{ color: string; valid: boolean }>();
+const { color, valid } = toRefs(props);
+const doneDisabled = computed(() => {
+  console.log('valid.value', valid.value);
+  return !valid.value;
 });
-const { color } = toRefs(props);
 
 const done = () => {
   emit('done');
@@ -40,6 +42,7 @@ const onDelete = () => {
               color="secondary"
               variant="elevated"
               @click="done"
+              :disabled="doneDisabled"
               data-testid="edit-label-done-button"
             >
               Done
diff --git a/src/components/ToolLabelEditor.vue b/src/components/ToolLabelEditor.vue
index c1e72c307..b047e3ff2 100644
--- a/src/components/ToolLabelEditor.vue
+++ b/src/components/ToolLabelEditor.vue
@@ -1,5 +1,5 @@
 <script setup lang="ts">
-import { computed } from 'vue';
+import { computed, ref } from 'vue';
 import LabelEditor from '@/src/components/LabelEditor.vue';
 import type { LabelsStore } from '../store/tools/useLabels';
 import type { AnnotationTool } from '../types/annotation-tool';
@@ -20,6 +20,8 @@ const props = defineProps<{
   labelsStore: LabelsStore<Pick<AnnotationTool, 'strokeWidth'>>;
 }>();
 
+const editingName = ref(props.name);
+
 const existingNames = computed(
   () =>
     new Set(
@@ -28,17 +30,28 @@ const existingNames = computed(
 );
 
 function isUniqueEditingName(name: string) {
-  return !existingNames.value.has(name);
+  console.log('isUniqueEditingName', name, !existingNames.value.has(name));
+  return !existingNames.value.has(name) || name === props.name;
 }
 
 function uniqueNameRule(name: string) {
   return isUniqueEditingName(name) || 'Name is not unique';
 }
+
+const valid = computed(() => {
+  console.log(
+    'valid',
+    editingName.value,
+    isUniqueEditingName(editingName.value)
+  );
+  return isUniqueEditingName(editingName.value);
+});
 </script>
 
 <template>
   <label-editor
     :color="color"
+    :valid="valid"
     @update:color="$emit('update:color', $event)"
     @cancel="$emit('cancel')"
     @done="$emit('done')"
@@ -53,7 +66,7 @@ function uniqueNameRule(name: string) {
       <v-text-field
         label="Name"
         class="flex-grow-0"
-        :model-value="name"
+        :model-value="editingName"
         @update:model-value="$emit('update:name', $event)"
         @keydown.stop.enter="done"
         :rules="[uniqueNameRule]"