Browse Source

fix: avoid unnecessary watch and use change event to simulate the same

pull/6466/head
Pranav C 1 year ago
parent
commit
379bf85d7f
  1. 8
      packages/nc-gui/components/smartsheet/toolbar/SearchData.vue

8
packages/nc-gui/components/smartsheet/toolbar/SearchData.vue

@ -63,13 +63,6 @@ const displayColumnLabel = computed(() => {
return columns.value?.find((column) => column.primaryValue)?.label return columns.value?.find((column) => column.primaryValue)?.label
}) })
watch(
() => search.value.field,
() => {
onPressEnter()
},
)
watchDebounced( watchDebounced(
() => search.value.query, () => search.value.query,
() => { () => {
@ -113,6 +106,7 @@ watch(columns, () => {
:dropdown-match-select-width="false" :dropdown-match-select-width="false"
dropdown-class-name="!rounded-lg nc-dropdown-toolbar-search-field-option w-48" dropdown-class-name="!rounded-lg nc-dropdown-toolbar-search-field-option w-48"
class="py-1 !absolute top-0 left-0 w-full h-full z-10 text-xs opacity-0" class="py-1 !absolute top-0 left-0 w-full h-full z-10 text-xs opacity-0"
@change="onPressEnter"
> >
<a-select-option v-for="op of columns" :key="op.value" :value="op.value"> <a-select-option v-for="op of columns" :key="op.value" :value="op.value">
<div class="text-[0.75rem] flex items-center -ml-1 gap-2"> <div class="text-[0.75rem] flex items-center -ml-1 gap-2">

Loading…
Cancel
Save