chore(gui): lint

Signed-off-by: Pranav C <pranavxc@gmail.com>

packages/nc-gui/components/smartsheet/Cell.vue

@@ -15,9 +15,9 @@ import {
   toRef,
   useColumn,
   useDebounceFn,
+  useSmartsheetRowStoreOrThrow,
   useVModel,
 } from '#imports'
-import { useSmartsheetRowStoreOrThrow } from '~/composables/useSmartsheetRowStore'
 import { NavigateDir } from '~/lib'
 
 interface Props {
@@ -65,7 +65,6 @@ const syncValue = useDebounceFn(function () {
   emit('save')
 }, 1000)
 
-
 const isAutoSaved = $computed(() => {
   return [
     UITypes.SingleLineText,

packages/nc-gui/components/smartsheet/Grid.vue

@@ -248,7 +248,8 @@ const onKeyDown = async (e: KeyboardEvent) => {
       e.preventDefault()
       if (selected.row < data.value.length - 1) selected.row++
       break
-    default: {
+    default:
+      {
         const rowObj = data.value[selected.row]
         const columnObj = fields.value[selected.col]
 
@@ -326,16 +327,21 @@ const showContextMenu = (e: MouseEvent, target?: { row: number; col: number }) =
 
 const rowRefs = $ref<any[]>()
 
+/** save/update records before unmounting the component */
 onBeforeUnmount(async () => {
   console.log(rowRefs)
-  for (const [index, currentRow] of Object.entries(data.value)) {
+  let index = -1
+  for (const currentRow of data.value) {
+    index++
+    /** if new record save row and save the LTAR cells */
     if (currentRow.rowMeta.new) {
       const syncLTARRefs = rowRefs[index]!.syncLTARRefs
-      const savedRow = await updateOrSaveRow(currentRow, null)
+      const savedRow = await updateOrSaveRow(currentRow, '')
       await syncLTARRefs(savedRow)
       currentRow.rowMeta.changed = false
       continue
     }
+    /** if existing row check updated cell and invoke update method */
     if (currentRow.rowMeta.changed) {
       currentRow.rowMeta.changed = false
       for (const field of meta?.value.columns ?? []) {
@@ -347,7 +353,6 @@ onBeforeUnmount(async () => {
     }
   }
 })
-
 </script>
 
 <template>
@@ -423,7 +428,7 @@ onBeforeUnmount(async () => {
             </tr>
           </thead>
           <tbody>
-          <SmartsheetRow ref="rowRefs" v-for="(row, rowIndex) of data" :key="rowIndex" :row="row">
+            <SmartsheetRow v-for="(row, rowIndex) of data" ref="rowRefs" :key="rowIndex" :row="row">
               <template #default="{ state }">
                 <tr class="nc-grid-row">
                   <td key="row-index" class="caption nc-grid-cell pl-5 pr-1">
@@ -443,8 +448,7 @@ onBeforeUnmount(async () => {
                         <a-checkbox v-model:checked="row.rowMeta.selected" />
                       </div>
                       <span class="flex-1" />
-                    <div v-if="!readOnly && !isLocked" class="nc-expand"
+                      <div v-if="!readOnly && !isLocked" class="nc-expand" :class="{ 'nc-comment': row.rowMeta?.commentCount }">
-                         :class="{ 'nc-comment': row.rowMeta?.commentCount }">
                         <span
                           v-if="row.rowMeta?.commentCount"
                           class="py-1 px-3 rounded-full text-xs cursor-pointer select-none transform hover:(scale-110)"
@@ -515,9 +519,9 @@ onBeforeUnmount(async () => {
             </SmartsheetRow>
 
             <!--
-    TODO: add relationType !== 'bt' ?
+TODO: add relationType !== 'bt' ?
-    v1: <tr v-if="!isView && !isLocked && !isPublicView && isEditable && relationType !== 'bt'">
+v1: <tr v-if="!isView && !isLocked && !isPublicView && isEditable && relationType !== 'bt'">
-  -->
+-->
             <tr v-if="!isView && !isLocked && isUIAllowed('xcDatatableEditable') && !isSqlView">
               <td
                 v-t="['c:row:add:grid-bottom']"

packages/nc-gui/components/smartsheet/Row.vue

@@ -35,7 +35,7 @@ reloadHook.on(() => {
 provide(ReloadRowDataHookInj, reloadHook)
 
 defineExpose({
-  syncLTARRefs
+  syncLTARRefs,
 })
 </script>
 

packages/nc-gui/components/smartsheet/expanded-form/index.vue

@@ -6,11 +6,11 @@ import Cell from '../Cell.vue'
 import VirtualCell from '../VirtualCell.vue'
 import Comments from './Comments.vue'
 import Header from './Header.vue'
-import { ReloadRowDataHookInj } from '~/context'
 import {
   FieldsInj,
   IsFormInj,
   MetaInj,
+  ReloadRowDataHookInj,
   computedInject,
   provide,
   ref,

packages/nc-gui/components/tabs/Smartsheet.vue

@@ -21,7 +21,6 @@ import {
 
 import type { TabItem } from '~/composables'
 
-
 const { activeTab } = defineProps<{
   activeTab: TabItem
 }>()
@@ -38,10 +37,7 @@ const fields = ref<ColumnType[]>([])
 //   TabMetaInj,
 //   computed(() => ({} as TabItem)),
 // )
-provide(
+provide(TabMetaInj, ref(activeTab))
-  TabMetaInj,
-  ref(activeTab),
-)
 const meta = computed<TableType>(() => metas.value?.[activeTab?.id as string])
 
 const reloadEventHook = createEventHook<void>()

packages/nc-gui/composables/useSmartsheetRowStore.ts

@@ -134,7 +134,7 @@ const [useProvideSmartsheetRowStore, useSmartsheetRowStore] = useInjectionState(
     removeLTARRef,
     syncLTARRefs,
     loadRow,
-    currentRow
+    currentRow,
   }
 }, 'smartsheet-row-store')
 

packages/nc-gui/composables/useViewData.ts

@@ -251,11 +251,11 @@ export function useViewData(
     }
   }
 
-  async function updateOrSaveRow(row: Row, property: string) {
+  async function updateOrSaveRow(row: Row, property?: string) {
     if (row.rowMeta.new) {
       return await insertRow(row.row, formattedData.value.indexOf(row))
     } else {
-      await updateRowProperty(row, property)
+      await updateRowProperty(row, property!)
     }
   }
 

packages/nc-gui/pages/[projectType]/[projectId]/index/index/[type]/[title]/[[viewTitle]].vue

@@ -1,6 +1,6 @@
 <script setup lang="ts">
-import { TabItem } from '~/composables'
+import type { TabItem } from '~/composables'
-import { TabMetaInj } from '~/context'
+import { TabMetaInj } from '#imports'
 
 const { getMeta } = useMetas()
 const route = useRoute()
@@ -20,7 +20,7 @@ getMeta(route.params.title as string, true).finally(() => {
   <div v-if="loading" class="flex items-center justify-center h-full w-full">
     <a-spin size="large" />
   </div>
-  <TabsSmartsheet :active-tab="activeTab" :key="route.params.title" v-else />
+  <TabsSmartsheet v-else :key="route.params.title" :active-tab="activeTab" />
 </template>
 
 <style scoped></style>