fix(nc-gui): stay on erd while table change

nocodb · Oct 3, 2023 · f6855d1 · f6855d1 · github-actions · Oct 3, 2023
1 parent 63e3b4d
commit f6855d1
Show file tree

Hide file tree

Showing 3 changed files with 47 additions and 9 deletions.
diff --git a/packages/nc-gui/components/dashboard/TreeView/ViewsList.vue b/packages/nc-gui/components/dashboard/TreeView/ViewsList.vue
@@ -192,6 +192,7 @@ async function changeView(view: ViewType) {
     tableId: table.value.id!,
     baseId: base.value.id!,
     hardReload: view.type === ViewTypes.FORM && selected.value[0] === view.id,
+    doNotSwitchTab: true,
   })
 
   if (isMobileMode.value) {

diff --git a/packages/nc-gui/composables/useTableNew.ts b/packages/nc-gui/composables/useTableNew.ts
@@ -49,6 +49,8 @@ export function useTableNew(param: { onTableCreate?: (tableMeta: TableType) => v
 
   const { loadTables, baseUrl, isXcdbBase } = useBase()
 
+  const { openedViewsTab, viewsByTable } = storeToRefs(useViewsStore())
+
   const workspaceId = computed(() => route.value.params.typeOrId as string)
 
   const tables = computed(() => baseTables.value.get(param.baseId) || [])
@@ -84,10 +86,18 @@ export function useTableNew(param: { onTableCreate?: (tableMeta: TableType) => v
       tableId: table.id,
     })
 
-    await navigateTo({
-      path: `/${workspaceIdOrType}/${baseIdOrBaseId}/${table?.id}`,
-      query: route.value.query,
-    })
+    const views = viewsByTable.value.get(table.id as string) ?? []
+
+    if (openedViewsTab.value !== 'view' && views[0].id) {
+      await navigateTo({
+        path: `/${workspaceIdOrType}/${baseIdOrBaseId}/${table?.id}/${views[0].id}/${openedViewsTab.value}`,
+        query: route.value.query,
+      })
+    } else
+      await navigateTo({
+        path: `/${workspaceIdOrType}/${baseIdOrBaseId}/${table?.id}`,
+        query: route.value.query,
+      })
 
     await getMeta(table.id as string)
   }

diff --git a/packages/nc-gui/store/views.ts b/packages/nc-gui/store/views.ts
@@ -154,46 +154,73 @@ export const useViewsStore = defineStore('viewsStore', () => {
     baseId,
     tableId,
     hardReload,
+    doNotSwitchTab,
   }: {
     view: ViewType
     baseId: string
     tableId: string
     hardReload?: boolean
+    doNotSwitchTab?: boolean
   }) => {
-    const routeName = 'index-typeOrId-baseId-index-index-viewId-viewTitle'
+    const routeName = 'index-typeOrId-baseId-index-index-viewId-viewTitle-slugs'
 
     let baseIdOrBaseId = baseId
 
     if (['base'].includes(route.value.params.typeOrId as string)) {
       baseIdOrBaseId = route.value.params.baseId as string
     }
 
+    const slugs = doNotSwitchTab ? router.currentRoute.value.params.slugs : undefined
+
     if (
       router.currentRoute.value.query &&
       router.currentRoute.value.query.page &&
       router.currentRoute.value.query.page === 'fields'
     ) {
       await router.push({
         name: routeName,
-        params: { viewTitle: view.id || '', viewId: tableId, baseId: baseIdOrBaseId },
+        params: {
+          viewTitle: view.id || '',
+          viewId: tableId,
+          baseId: baseIdOrBaseId,
+          slugs,
+        },
         query: router.currentRoute.value.query,
       })
     } else {
-      await router.push({ name: routeName, params: { viewTitle: view.id || '', viewId: tableId, baseId: baseIdOrBaseId } })
+      await router.push({
+        name: routeName,
+        params: {
+          viewTitle: view.id || '',
+          viewId: tableId,
+          baseId: baseIdOrBaseId,
+          slugs,
+        },
+      })
     }
 
     if (hardReload) {
       await router
         .replace({
           name: routeName,
           query: { reload: 'true' },
-          params: { viewId: tableId, baseId: baseIdOrBaseId, viewTitle: view.id || '' },
+          params: {
+            viewId: tableId,
+            baseId: baseIdOrBaseId,
+            viewTitle: view.id || '',
+            slugs,
+          },
         })
         .then(() => {
           router.replace({
             name: routeName,
             query: {},
-            params: { viewId: tableId, viewTitle: view.id || '', baseId: baseIdOrBaseId },
+            params: {
+              viewId: tableId,
+              viewTitle: view.id || '',
+              baseId: baseIdOrBaseId,
+              slugs,
+            },
           })
         })
     }