Merge pull request #6860 from LedgerHQ/fix/LIVE-12568

fix(lld): auto scroll when mounted on accounts page
LedgerHQ · May 16, 2024 · 2e989ad · 2e989ad
2 parents 8a065af + 1364fe0
commit 2e989ad
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 3 deletions.
diff --git a/.changeset/nice-trains-dream.md b/.changeset/nice-trains-dream.md
@@ -0,0 +1,5 @@
+---
+"ledger-live-desktop": patch
+---
+
+fix automatic scroll that was triggered on accounts page when mounted
diff --git a/apps/ledger-live-desktop/src/renderer/screens/accounts/AccountRowItem/index.tsx b/apps/ledger-live-desktop/src/renderer/screens/accounts/AccountRowItem/index.tsx
@@ -145,7 +145,13 @@ const AccountRowItem = (props: Props) => {
 
   const scrollTopFocusRef = useRef<HTMLSpanElement>(null);
 
+  const mountedRef = useRef(true);
+
   useEffect(() => {
+    if (mountedRef.current) {
+      mountedRef.current = false;
+      return;
+    }
     if (scrollTopFocusRef.current && !expanded) {
       scrollTopFocusRef.current.scrollIntoView({
         block: "nearest",
@@ -160,9 +166,7 @@ const AccountRowItem = (props: Props) => {
     setExpanded(expandedStates[account.id]);
   };
 
-  const onClickHandler = () => {
-    onClick(account, parentAccount);
-  };
+  const onClickHandler = () => onClick(account, parentAccount);
 
   const unit = useAccountUnit(account);