From 41b9971cf3d2f30fa32121fd3095117070c84203 Mon Sep 17 00:00:00 2001
From: Muhammed Mustafa <mustafa010199@gmail.com>
Date: Thu, 18 Aug 2022 15:04:00 +0530
Subject: [PATCH] fix/pr review included

---
 .../components/smartsheet/sidebar/toolbar/AddRow.vue   | 10 ++++++----
 packages/nc-gui-v2/composables/useViewData.ts          |  2 ++
 2 files changed, 8 insertions(+), 4 deletions(-)

diff --git a/packages/nc-gui-v2/components/smartsheet/sidebar/toolbar/AddRow.vue b/packages/nc-gui-v2/components/smartsheet/sidebar/toolbar/AddRow.vue
index 4aed551e22..dffca10e5e 100644
--- a/packages/nc-gui-v2/components/smartsheet/sidebar/toolbar/AddRow.vue
+++ b/packages/nc-gui-v2/components/smartsheet/sidebar/toolbar/AddRow.vue
@@ -1,9 +1,14 @@
 <script setup lang="ts">
 import { OpenNewRecordFormHookInj, inject } from '#imports'
+
 const { isOpen } = useSidebar({ storageKey: 'nc-right-sidebar' })
 const isLocked = inject(IsLockedInj)
 
 const openNewRecordFormHook = inject(OpenNewRecordFormHookInj)!
+
+const onClick = () => {
+  if (!isLocked) openNewRecordFormHook.trigger()
+}
 </script>
 
 <template>
@@ -13,10 +18,7 @@ const openNewRecordFormHook = inject(OpenNewRecordFormHookInj)!
       :class="{ 'hover:after:bg-primary/75 group': !isLocked, 'disabled-ring': isLocked }"
       class="nc-sidebar-right-item nc-sidebar-add-row"
     >
-      <MdiPlusOutline
-        :class="{ 'cursor-pointer group-hover:(!text-white)': !isLocked, 'disabled': isLocked }"
-        @click="!isLocked ? openNewRecordFormHook.trigger() : {}"
-      />
+      <MdiPlusOutline :class="{ 'cursor-pointer group-hover:(!text-white)': !isLocked, 'disabled': isLocked }" @click="onClick" />
     </div>
   </a-tooltip>
 </template>
diff --git a/packages/nc-gui-v2/composables/useViewData.ts b/packages/nc-gui-v2/composables/useViewData.ts
index a0d3390b7a..70eac6ceda 100644
--- a/packages/nc-gui-v2/composables/useViewData.ts
+++ b/packages/nc-gui-v2/composables/useViewData.ts
@@ -214,11 +214,13 @@ export function useViewData(
       oldRow: {},
       rowMeta: { new: true },
     })
+
     return formattedData.value[addAfter]
   }
 
   const removeLastEmptyRow = () => {
     const lastRow = formattedData.value[formattedData.value.length - 1]
+
     if (lastRow.rowMeta.new) {
       formattedData.value.pop()
     }