From 3ccf1dfd39e4b8d6000ac4b11bdba9837e109c6f Mon Sep 17 00:00:00 2001
From: Pranav C <pranavxc@gmail.com>
Date: Wed, 10 Jan 2024 09:05:29 +0000
Subject: [PATCH] fix: keep the element reference and reset scrollTop on that

---
 .../components/smartsheet/expanded-form/index.vue      | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/packages/nc-gui/components/smartsheet/expanded-form/index.vue b/packages/nc-gui/components/smartsheet/expanded-form/index.vue
index 5c8d0c9b4b..7f7f06c6a6 100644
--- a/packages/nc-gui/components/smartsheet/expanded-form/index.vue
+++ b/packages/nc-gui/components/smartsheet/expanded-form/index.vue
@@ -469,12 +469,14 @@ const isReadOnlyVirtualCell = (column: ColumnType) => {
 watch([expandedFormScrollWrapper, isLoading], () => {
   if (isMobileMode.value) return
 
-  if (expandedFormScrollWrapper.value && !isLoading.value) {
-    const height = expandedFormScrollWrapper.value.scrollHeight
-    expandedFormScrollWrapper.value.scrollTop = height
+  const expandedFormScrollWrapperEl = expandedFormScrollWrapper.value
+
+  if (expandedFormScrollWrapperEl && !isLoading.value) {
+    const height = expandedFormScrollWrapperEl.scrollHeight
+    expandedFormScrollWrapperEl.scrollTop = height
 
     setTimeout(() => {
-      expandedFormScrollWrapper.value.scrollTop = 0
+      expandedFormScrollWrapperEl.scrollTop = 0
     }, 125)
   }
 })