From 76b47597f8877df31e52f724a716bfa518a37fa2 Mon Sep 17 00:00:00 2001
From: Raju Udava <sivadstala@gmail.com>
Date: Wed, 24 Nov 2021 17:17:32 +0530
Subject: [PATCH] revert: previous merge

Signed-off-by: Raju Udava <sivadstala@gmail.com>
---
 .../nc-gui/components/ProjectTreeView.vue     | 75 +++++--------------
 .../project/spreadsheet/rowsXcDataTable.vue   | 10 +--
 packages/nc-gui/components/project/table.vue  |  4 +-
 packages/nc-gui/components/projectTabs.vue    |  2 +-
 .../nocodb/src/lib/noco/meta/NcMetaMgr.ts     |  4 +-
 5 files changed, 24 insertions(+), 71 deletions(-)

diff --git a/packages/nc-gui/components/ProjectTreeView.vue b/packages/nc-gui/components/ProjectTreeView.vue
index f53a8d887b..100c3e2c07 100644
--- a/packages/nc-gui/components/ProjectTreeView.vue
+++ b/packages/nc-gui/components/ProjectTreeView.vue
@@ -26,24 +26,16 @@
                   &lt;!&ndash;        <v-icon v-else>mdi-arrow-expand-horizontal</v-icon>&ndash;&gt;
 
                 </v-btn>-->
-          <v-text-field
-            v-model="search"
-            placeholder="Search tables"
-            dense
-            hide-details
-            class="elevation-0 mr-2  pl-3 pr-1 caption nc-table-list-filter"
-          >
-            <template #prepend-inner>
-              <v-icon small class="mt-2 ml-2 mr-1 ">
-                mdi-magnify
-              </v-icon>
-            </template>
-            <template #append>
-              <v-icon v-if="search" class="mt-3 mr-3" color="grey" x-small @click="search=''">
-                mdi-close
-              </v-icon>
-            </template>
-          </v-text-field>
+          <!--      <v-text-field-->
+          <!--        v-model="search"-->
+          <!--        label="Search Project"-->
+          <!--        dense-->
+          <!--        solo-->
+          <!--        hide-details-->
+          <!--        clearable-->
+          <!--        clear-icon="mdi-close-circle-outline"-->
+          <!--        class="pa-2"-->
+          <!--      ></v-text-field>-->
 
           <v-skeleton-loader
             v-if="!projects || !projects.length"
@@ -116,7 +108,7 @@
               </v-tooltip>
             </template>
           </v-treeview>
-          <v-container v-else fluid class="px-1 pt-0">
+          <v-container v-else fluid class="px-1">
             <v-list dense expand class="nc-project-tree nc-single-env-project-tree">
               <template v-for="item in listViewArr">
                 <!--                   v-if="item.children && item.children.length"-->
@@ -124,7 +116,7 @@
                   v-if="isNested(item) && showNode(item)"
                   :key="item.type"
                   color="textColor"
-                  :value="isActiveList(item) || search"
+                  :value="isActiveList(item)"
                   @click="
                     !(item.children && item.children.length) && addTab({ ...item }, false, false)
                   "
@@ -139,7 +131,7 @@
                     <v-list-item-icon>
                       <v-icon
                         v-if="open && icons[item._nodes.type].openIcon"
-                        small
+                        x-small
                         style="cursor: auto"
                         :color="icons[item._nodes.type].openColor"
                       >
@@ -147,7 +139,7 @@
                       </v-icon>
                       <v-icon
                         v-else
-                        small
+                        x-small
                         style="cursor: auto"
                         :color="icons[item._nodes.type].color"
                       >
@@ -157,25 +149,17 @@
                     <v-list-item-title>
                       <v-tooltip v-if="!isNonAdminAccessAllowed(item)" top>
                         <template #activator="{ on }">
-                          <span v-if="item.type === 'tableDir'" class="caption font-weight-regular" v-on="on">
-                            Tables<template v-if="item.children && item.children.length"> ({{
-                              item.children.filter(child => !search || child.name.toLowerCase().includes(search.toLowerCase())).length
-                            }})</template></span>
-                          <span v-else class="caption font-weight-regular" v-on="on">
+                          <span class="caption font-weight-regular" v-on="on">
                             {{ item.name }}</span>
                         </template>
                         <span class="caption">Only visible to Creator</span>
                       </v-tooltip>
-                      <template
+                      <span
                         v-else
+                        class="caption font-weight-regular"
+                        @dblclick="showSqlClient = true"
                       >
-                        <span v-if="item.type === 'tableDir'" class="caption font-weight-regular" v-on="on">
-                          Tables<template v-if="item.children && item.children.length"> ({{
-                            item.children.filter(child => !search || child.name.toLowerCase().includes(search.toLowerCase())).length
-                          }})</template></span>
-                        <span v-else class="caption font-weight-regular" v-on="on">
-                          {{ item.name }}</span>
-                      </template>
+                        {{ item.name }}</span>
                     </v-list-item-title>
 
                     <v-spacer />
@@ -202,7 +186,6 @@
                   <v-list-item-group :value="selectedItem">
                     <v-list-item
                       v-for="child in item.children || []"
-                      v-show="!search || child.name.toLowerCase().includes(search.toLowerCase())"
                       :key="child.key"
                       color="x-active"
                       active-class="font-weight-bold"
@@ -730,7 +713,7 @@ export default {
     open: [],
     search: null,
     menuVisible: false,
-    excelImportDialog: false,
+    excelImportDialog:false,
     x: 0,
     y: 0,
     menuItem: null,
@@ -1785,24 +1768,6 @@ export default {
 .nested:hover .action {
   opacity: 1;
 }
-
-/deep/ .nc-table-list-filter .v-input__slot {
-  min-height: 30px !important;
-}
-/deep/ .nc-table-list-filter .v-input__slot label {
-  top: 6px;
-}
-
-
-
-/deep/ .nc-table-list-filter.theme--light.v-text-field > .v-input__control > .v-input__slot:before {
-  border-top-color: rgba(0, 0, 0, 0.12) !important;
-}
-
-/deep/ .nc-table-list-filter.theme--dark.v-text-field > .v-input__control > .v-input__slot:before {
-  border-top-color: rgba(255, 255, 255, 0.12) !important;
-}
-
 </style>
 
 <!--
diff --git a/packages/nc-gui/components/project/spreadsheet/rowsXcDataTable.vue b/packages/nc-gui/components/project/spreadsheet/rowsXcDataTable.vue
index 2ffee39118..6be60c7fd3 100644
--- a/packages/nc-gui/components/project/spreadsheet/rowsXcDataTable.vue
+++ b/packages/nc-gui/components/project/spreadsheet/rowsXcDataTable.vue
@@ -583,7 +583,6 @@ export default {
   },
   mixins: [spreadsheet],
   props: {
-    isActive: Boolean,
     tabId: String,
     env: String,
     nodes: Object,
@@ -671,11 +670,6 @@ export default {
     rowContextMenu: null
   }),
   watch: {
-    isActive(n, o) {
-      if (!o && n) {
-        this.reload()
-      }
-    },
     page(p) {
       this.$store.commit('tabs/MutSetTabState', {
         id: this.uniqueId,
@@ -1165,9 +1159,7 @@ export default {
     }
   },
   computed: {
-    tabsState() {
-      return this.$store.state.tabs.tabsState || {}
-    },
+    tabsState() { return this.$store.state.tabs.tabsState || {} },
     uniqueId() {
       return `${this.tabId}_${this.selectedViewId}`
     },
diff --git a/packages/nc-gui/components/project/table.vue b/packages/nc-gui/components/project/table.vue
index 3a0ea1c9d3..29e3090256 100644
--- a/packages/nc-gui/components/project/table.vue
+++ b/packages/nc-gui/components/project/table.vue
@@ -317,7 +317,6 @@
           >
             <rows-xc-data-table
               ref="tabs7"
-              :is-active="isActive"
               :tab-id="tabId"
               :show-tabs="relationTabs && relationTabs.length"
               :table="nodes.tn"
@@ -385,7 +384,6 @@
             <template>
               <rows-xc-data-table
                 ref="tabs7"
-                :is-active="isActive"
                 :show-tabs="relationTabs && relationTabs.length"
                 :table="nodes.tn"
                 :nodes="nodes"
@@ -641,7 +639,7 @@ export default {
   head() {
     return {}
   },
-  props: ['nodes', 'hideLogWindows', 'tabId', 'isActive']
+  props: ['nodes', 'hideLogWindows', 'tabId']
 }
 </script>
 
diff --git a/packages/nc-gui/components/projectTabs.vue b/packages/nc-gui/components/projectTabs.vue
index bab9ad1370..89cbd7d27e 100644
--- a/packages/nc-gui/components/projectTabs.vue
+++ b/packages/nc-gui/components/projectTabs.vue
@@ -61,8 +61,8 @@
           >
             <!--          <sqlLogAndOutput :hide="hideLogWindows">-->
             <TableView
+              v-if="activeTab === `${(tab._nodes && tab._nodes).type || ''}||${(tab._nodes && tab._nodes.dbAlias) || ''}||${tab.name}`"
               :ref="'tabs'+index"
-              :is-active="activeTab === `${(tab._nodes && tab._nodes).type || ''}||${(tab._nodes && tab._nodes.dbAlias) || ''}||${tab.name}`"
               :tab-id="`${pid}||${(tab._nodes && tab._nodes).type || ''}||${(tab._nodes && tab._nodes.dbAlias) || ''}||${tab.name}`"
               :hide-log-windows.sync="hideLogWindows"
               :nodes="tab._nodes"
diff --git a/packages/nocodb/src/lib/noco/meta/NcMetaMgr.ts b/packages/nocodb/src/lib/noco/meta/NcMetaMgr.ts
index 9e7d8310ee..b181164a09 100644
--- a/packages/nocodb/src/lib/noco/meta/NcMetaMgr.ts
+++ b/packages/nocodb/src/lib/noco/meta/NcMetaMgr.ts
@@ -4568,9 +4568,7 @@ export default class NcMetaMgr {
               result[d.title].disabled[d.role] = !!d.disabled;
             }
 
-            return Object.values(result)?.sort((a: any, b: any) =>
-              (a?._tn || a?.tn)?.localeCompare(b?._tn || b?.tn)
-            );
+            return Object.values(result);
           }
           break;
         case 'view':