Merge pull request #252 from eficode/feature/fix-estimation-propagation

fix(app-board): update estimation label
eficode · Mar 1, 2021 · cb90e9b · cb90e9b
2 parents 694fd63 + 2380679
commit cb90e9b
Showing 1 changed file with 8 additions and 2 deletions.
diff --git a/packages/game-app/src/_shared/components/EstimationEditor/EstimationEditor.tsx b/packages/game-app/src/_shared/components/EstimationEditor/EstimationEditor.tsx
@@ -54,8 +54,14 @@ const EstimationEditor: React.FC<Props> = ({
   );
 
   useEffect(() => {
-    inputRef?.current?.focus();
-  }, []);
+    setEstimation(initialEstimation || '');
+  }, [initialEstimation]);
+
+  useEffect(() => {
+    if (state === 'open') {
+      inputRef?.current?.focus();
+    }
+  }, [state]);
 
   return (
     <EstimationWrapper state={state} moving={moving} chars={estimation ? estimation.length : 0}>