[Snapshot Restore] Disable steps when form is invalid (elastic#76540)

alisonelizabeth · Sep 14, 2020 · 0e02da1 · 0e02da1
1 parent 23eaf58
commit 0e02da1
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/x-pack/plugins/snapshot_restore/public/application/components/policy_form/navigation.tsx b/x-pack/plugins/snapshot_restore/public/application/components/policy_form/navigation.tsx
@@ -11,12 +11,14 @@ interface Props {
   currentStep: number;
   maxCompletedStep: number;
   updateCurrentStep: (step: number) => void;
+  isFormValid: boolean;
 }
 
 export const PolicyNavigation: React.FunctionComponent<Props> = ({
   currentStep,
   maxCompletedStep,
   updateCurrentStep,
+  isFormValid,
 }) => {
   const { i18n } = useServices();
 
@@ -27,6 +29,7 @@ export const PolicyNavigation: React.FunctionComponent<Props> = ({
       }),
       isComplete: maxCompletedStep >= 1,
       isSelected: currentStep === 1,
+      disabled: !isFormValid && currentStep !== 1,
       onClick: () => updateCurrentStep(1),
     },
     {
@@ -35,7 +38,7 @@ export const PolicyNavigation: React.FunctionComponent<Props> = ({
       }),
       isComplete: maxCompletedStep >= 2,
       isSelected: currentStep === 2,
-      disabled: maxCompletedStep < 1,
+      disabled: maxCompletedStep < 1 || (!isFormValid && currentStep !== 2),
       onClick: () => updateCurrentStep(2),
     },
     {
@@ -44,7 +47,7 @@ export const PolicyNavigation: React.FunctionComponent<Props> = ({
       }),
       isComplete: maxCompletedStep >= 3,
       isSelected: currentStep === 3,
-      disabled: maxCompletedStep < 2,
+      disabled: maxCompletedStep < 2 || (!isFormValid && currentStep !== 3),
       onClick: () => updateCurrentStep(3),
     },
     {
@@ -53,7 +56,7 @@ export const PolicyNavigation: React.FunctionComponent<Props> = ({
       }),
       isComplete: maxCompletedStep >= 3,
       isSelected: currentStep === 4,
-      disabled: maxCompletedStep < 3,
+      disabled: maxCompletedStep < 3 || (!isFormValid && currentStep !== 4),
       onClick: () => updateCurrentStep(4),
     },
   ];

diff --git a/x-pack/plugins/snapshot_restore/public/application/components/policy_form/policy_form.tsx b/x-pack/plugins/snapshot_restore/public/application/components/policy_form/policy_form.tsx
@@ -130,6 +130,7 @@ export const PolicyForm: React.FunctionComponent<Props> = ({
         currentStep={currentStep}
         maxCompletedStep={maxCompletedStep}
         updateCurrentStep={updateCurrentStep}
+        isFormValid={validation.isValid}
       />
       <EuiSpacer size="l" />
       <EuiForm>