From 29f024a498b65d860d7ac5beec7c6ac190882ad6 Mon Sep 17 00:00:00 2001
From: Raju Udava <86527202+dstala@users.noreply.github.com>
Date: Mon, 11 Sep 2023 22:28:17 +0530
Subject: [PATCH] refactor: placeholder correction

Signed-off-by: Raju Udava <86527202+dstala@users.noreply.github.com>
---
 packages/nc-gui/components/cell/DatePicker.vue     | 2 +-
 packages/nc-gui/components/cell/DateTimePicker.vue | 2 +-
 packages/nc-gui/components/cell/MultiSelect.vue    | 2 +-
 packages/nc-gui/components/cell/SingleSelect.vue   | 2 +-
 packages/nc-gui/components/cell/TimePicker.vue     | 2 +-
 packages/nc-gui/components/cell/YearPicker.vue     | 2 +-
 6 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/packages/nc-gui/components/cell/DatePicker.vue b/packages/nc-gui/components/cell/DatePicker.vue
index dec3b379b8..738bffc8d8 100644
--- a/packages/nc-gui/components/cell/DatePicker.vue
+++ b/packages/nc-gui/components/cell/DatePicker.vue
@@ -84,7 +84,7 @@ watch(
 
 const placeholder = computed(() => {
   if (isEditColumn.value && modelValue === null) {
-    return 'Select default date (Optional)'
+    return '(Optional)'
   } else if (modelValue === null && showNull.value) {
     return 'NULL'
   } else if (isDateInvalid.value) {
diff --git a/packages/nc-gui/components/cell/DateTimePicker.vue b/packages/nc-gui/components/cell/DateTimePicker.vue
index a64636a64b..9324dedd52 100644
--- a/packages/nc-gui/components/cell/DateTimePicker.vue
+++ b/packages/nc-gui/components/cell/DateTimePicker.vue
@@ -137,7 +137,7 @@ watch(
 
 const placeholder = computed(() => {
   if (isEditColumn.value && modelValue === null) {
-    return 'Select default date (Optional)'
+    return '(Optional)'
   } else if (modelValue === null && showNull.value) {
     return 'NULL'
   } else if (isDateInvalid.value) {
diff --git a/packages/nc-gui/components/cell/MultiSelect.vue b/packages/nc-gui/components/cell/MultiSelect.vue
index f7fb74bc55..c22768c007 100644
--- a/packages/nc-gui/components/cell/MultiSelect.vue
+++ b/packages/nc-gui/components/cell/MultiSelect.vue
@@ -387,7 +387,7 @@ const selectedOpts = computed(() => {
       v-model:value="vModel"
       mode="multiple"
       class="w-full overflow-hidden"
-      :placeholder="isEditColumn ? 'Select default value (Optional)' : ''"
+      :placeholder="isEditColumn ? '(Optional)' : ''"
       :bordered="false"
       clear-icon
       show-search
diff --git a/packages/nc-gui/components/cell/SingleSelect.vue b/packages/nc-gui/components/cell/SingleSelect.vue
index afaa0e7aa6..3fce42de11 100644
--- a/packages/nc-gui/components/cell/SingleSelect.vue
+++ b/packages/nc-gui/components/cell/SingleSelect.vue
@@ -303,7 +303,7 @@ const selectedOpt = computed(() => {
       v-model:value="vModel"
       class="w-full overflow-hidden"
       :class="{ 'caret-transparent': !hasEditRoles }"
-      :placeholder="isEditColumn ? 'Select default option (Optional)' : ''"
+      :placeholder="isEditColumn ? '(Optional)' : ''"
       :allow-clear="!column.rqd && editAllowed"
       :bordered="false"
       :open="isOpen && editAllowed"
diff --git a/packages/nc-gui/components/cell/TimePicker.vue b/packages/nc-gui/components/cell/TimePicker.vue
index 0428de9d07..31bb12a6ad 100644
--- a/packages/nc-gui/components/cell/TimePicker.vue
+++ b/packages/nc-gui/components/cell/TimePicker.vue
@@ -89,7 +89,7 @@ watch(
 
 const placeholder = computed(() => {
   if (isEditColumn.value && modelValue === null) {
-    return 'Select default time (Optional)'
+    return '(Optional)'
   } else if (modelValue === null && showNull.value) {
     return 'NULL'
   } else if (isTimeInvalid.value) {
diff --git a/packages/nc-gui/components/cell/YearPicker.vue b/packages/nc-gui/components/cell/YearPicker.vue
index e2e096eb99..5ba1d252ae 100644
--- a/packages/nc-gui/components/cell/YearPicker.vue
+++ b/packages/nc-gui/components/cell/YearPicker.vue
@@ -76,7 +76,7 @@ watch(
 
 const placeholder = computed(() => {
   if (isEditColumn.value && modelValue === null) {
-    return 'Select default year (Optional)'
+    return '(Optional)'
   } else if (modelValue === null && showNull.value) {
     return 'NULL'
   } else if (isYearInvalid.value) {