From 3ef91156743ed251f0d992ac69d5dd3302a96c27 Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Mon, 10 Oct 2022 12:21:02 +0530
Subject: [PATCH] refactor(gui): avoid locking treeview operation based on view
 lock

Signed-off-by: Pranav C <pranavxc@gmail.com>
---
 packages/nc-gui/components/dashboard/TreeView.vue         | 7 ++-----
 packages/nc-gui/components/tabs/Smartsheet.vue            | 6 ------
 packages/nc-gui/pages/[projectType]/[projectId]/index.vue | 5 -----
 3 files changed, 2 insertions(+), 16 deletions(-)

diff --git a/packages/nc-gui/components/dashboard/TreeView.vue b/packages/nc-gui/components/dashboard/TreeView.vue
index 14c7e8d551..26c9bf60c1 100644
--- a/packages/nc-gui/components/dashboard/TreeView.vue
+++ b/packages/nc-gui/components/dashboard/TreeView.vue
@@ -5,7 +5,6 @@ import GithubButton from 'vue-github-button'
 import {
   Empty,
   computed,
-  inject,
   reactive,
   ref,
   resolveComponent,
@@ -36,8 +35,6 @@ const { isUIAllowed } = useUIPermission()
 
 const [searchActive, toggleSearchActive] = useToggle()
 
-const isLocked = inject('TreeViewIsLockedInj')
-
 let key = $ref(0)
 
 const menuRef = $ref<HTMLLIElement>()
@@ -349,7 +346,7 @@ function openTableCreateDialog() {
                     </div>
 
                     <a-dropdown
-                      v-if="!isSharedBase && !isLocked && (isUIAllowed('table-rename') || isUIAllowed('table-delete'))"
+                      v-if="!isSharedBase && (isUIAllowed('table-rename') || isUIAllowed('table-delete'))"
                       :trigger="['click']"
                       @click.stop
                     >
@@ -383,7 +380,7 @@ function openTableCreateDialog() {
         </div>
       </div>
 
-      <template v-if="!isLocked && !isSharedBase" #overlay>
+      <template v-if="!isSharedBase" #overlay>
         <a-menu class="!py-0 rounded text-sm">
           <template v-if="contextMenuTarget.type === 'table'">
             <a-menu-item v-if="isUIAllowed('table-rename')" @click="openRenameTableDialog(contextMenuTarget.value, true)">
diff --git a/packages/nc-gui/components/tabs/Smartsheet.vue b/packages/nc-gui/components/tabs/Smartsheet.vue
index 9d743b152a..76fd70f60b 100644
--- a/packages/nc-gui/components/tabs/Smartsheet.vue
+++ b/packages/nc-gui/components/tabs/Smartsheet.vue
@@ -12,14 +12,12 @@ import {
   TabMetaInj,
   computed,
   createEventHook,
-  inject,
   provide,
   ref,
   toRef,
   useMetas,
   useProvideKanbanViewStore,
   useProvideSmartsheetStore,
-  watch,
 } from '#imports'
 import type { TabItem } from '~/lib'
 
@@ -57,10 +55,6 @@ provide(OpenNewRecordFormHookInj, openNewRecordFormHook)
 provide(FieldsInj, fields)
 provide(IsFormInj, isForm)
 provide(TabMetaInj, activeTab)
-
-const treeViewIsLockedInj = inject('TreeViewIsLockedInj', ref(false))
-
-watch(isLocked, (nextValue) => (treeViewIsLockedInj.value = nextValue), { immediate: true })
 </script>
 
 <template>
diff --git a/packages/nc-gui/pages/[projectType]/[projectId]/index.vue b/packages/nc-gui/pages/[projectType]/[projectId]/index.vue
index 44ee2edbba..e37fac58bd 100644
--- a/packages/nc-gui/pages/[projectType]/[projectId]/index.vue
+++ b/packages/nc-gui/pages/[projectType]/[projectId]/index.vue
@@ -11,7 +11,6 @@ import {
   onMounted,
   openLink,
   projectThemeColors,
-  provide,
   ref,
   useCopy,
   useGlobal,
@@ -48,10 +47,6 @@ const { isUIAllowed } = useUIPermission()
 
 const { copy } = useCopy()
 
-const isLocked = ref(false)
-
-provide('TreeViewIsLockedInj', isLocked)
-
 // create a new sidebar state
 const { isOpen, toggle, toggleHasSidebar } = useSidebar('nc-left-sidebar', { hasSidebar: false, isOpen: false })