refactor: avoid setPreselected many times

qlik-oss · Mar 9, 2023 · cc91d16 · cc91d16
1 parent f47a7ba
commit cc91d16
Showing 1 changed file with 6 additions and 2 deletions.
diff --git a/apis/nucleus/src/components/listbox/hooks/selections/useSelectionsInteractions.js b/apis/nucleus/src/components/listbox/hooks/selections/useSelectionsInteractions.js
@@ -79,7 +79,9 @@ export default function useSelectionsInteractions({
         return;
       }
       const elemNumber = +event.currentTarget.getAttribute('data-n');
-      setPreSelected([elemNumber]);
+      if (!(preSelected.length === 1 && preSelected[0] === elemNumber)) {
+        setPreSelected([elemNumber]);
+      }
       handleSingleSelectKey(event);
     },
     [selectingValues, selectDisabled]
@@ -94,7 +96,9 @@ export default function useSelectionsInteractions({
       setMouseDown(true);
 
       const elemNumber = +event.currentTarget.getAttribute('data-n');
-      setPreSelected([elemNumber]);
+      if (!(preSelected.length === 1 && preSelected[0] === elemNumber)) {
+        setPreSelected([elemNumber]);
+      }
       handleSingleSelectKey(event);
     },
     [selectingValues, selectDisabled]