fix(Wizard): allow dynamic step update (#10748)

patternfly · Jul 16, 2024 · 9257008 · 9257008
1 parent c16c6f4
commit 9257008
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/packages/react-core/src/components/Wizard/Wizard.tsx b/packages/react-core/src/components/Wizard/Wizard.tsx
@@ -91,6 +91,11 @@ export const Wizard = ({
     }
   }, [startIndex]);
 
+  // When children change, active step index should reset
+  React.useEffect(() => {
+    setActiveStepIndex(startIndex);
+  }, [children, startIndex]);
+
   const focusMainContentElement = () =>
     setTimeout(() => {
       wrapperRef?.current?.focus && wrapperRef.current.focus();

diff --git a/packages/react-core/src/components/Wizard/WizardContext.tsx b/packages/react-core/src/components/Wizard/WizardContext.tsx
@@ -74,6 +74,11 @@ export const WizardContextProvider: React.FunctionComponent<WizardContextProvide
   const [currentSteps, setCurrentSteps] = React.useState<WizardStepType[]>(initialSteps);
   const [currentFooter, setCurrentFooter] = React.useState<WizardFooterType>();
 
+  // Callback to update steps if they change after initial render
+  React.useEffect(() => {
+    setCurrentSteps(initialSteps);
+  }, [initialSteps]);
+
   // Combined initial and current state steps
   const steps = React.useMemo(
     () =>