fix trackingEffects bug when the first value could not be selected (#37)

Bloomca · Jun 4, 2024 · cf6fcdb · cf6fcdb
1 parent fff0896
commit cf6fcdb
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/src/hooks/combine-state.ts b/src/hooks/combine-state.ts
@@ -84,7 +84,7 @@ export function combineState(...states) {
       // it is guaranteed that reading `state.getValue` will
       // return the updated value
       const updatedValue = states.map((state) => state.getValue());
-      combinedState.setValue(() => updatedValue);
+      combinedState.setValue(updatedValue);
     });
   });
 

diff --git a/src/hooks/create-state.ts b/src/hooks/create-state.ts
@@ -436,6 +436,8 @@ function createState<T>(
         }
 
         cb(newSelectedValue);
+        // update selected value
+        trackingEffect.selectedValue = newSelectedValue;
       });
 
       trackingIterators.forEach((trackingIterator) => {