Modify stepper to be linear

frontend/src/components/Stepper.js

@@ -2,18 +2,13 @@ import * as React from 'react';
 import Box from '@mui/material/Box';
 import Stepper from '@mui/material/Stepper';
 import Step from '@mui/material/Step';
-import StepButton from '@mui/material/StepButton';
+import StepLabel from '@mui/material/StepLabel';
 import Button from '@mui/material/Button';
-import Typography from '@mui/material/Typography';
 
 import LocationStep from './steps/Location'
 import SoilStep from './steps/Soil'
 import ResultsStep from './steps/Results'
 
-export default function StepperWizard(props) {
-  const [activeStep, setActiveStep] = React.useState(0);
-  const [completed, setCompleted] = React.useState({});
-
 const steps = [
     {'label': 'Select location', 'component': LocationStep },
     {'label': 'Choose soil', 'component': SoilStep },
@@ -23,67 +18,35 @@ export default function StepperWizard(props) {
     {'label': 'Summary', 'component': SoilStep }
 ];
 
-  const totalSteps = () => {
-    return steps.length;
-  };
-
-  const completedSteps = () => {
-    return Object.keys(completed).length;
-  };
-
-  const isLastStep = () => {
-    return activeStep === totalSteps() - 1;
-  };
-
-  const allStepsCompleted = () => {
-    return completedSteps() === totalSteps();
-  };
+export default function StepperWizard(props) {
+  const [activeStep, setActiveStep] = React.useState(0);
 
   const handleNext = () => {
-    const newActiveStep =
-      isLastStep() && !allStepsCompleted()
-        ? // It's the last step, but not all steps have been completed,
-          // find the first step that has been completed
-          steps.findIndex((step, i) => !(i in completed))
-        : activeStep + 1;
-    setActiveStep(newActiveStep);
+    setActiveStep((prevActiveStep) => prevActiveStep + 1);
   };
 
   const handleBack = () => {
     setActiveStep((prevActiveStep) => prevActiveStep - 1);
   };
 
-  const handleStep = (step) => () => {
-    setActiveStep(step);
-  };
-
-  const handleComplete = () => {
-    const newCompleted = completed;
-    newCompleted[activeStep] = true;
-    setCompleted(newCompleted);
-    handleNext();
-  };
-
   const handleReset = () => {
     setActiveStep(0);
-    setCompleted({});
   };
 
-  let CurrentStep = activeStep > totalSteps() ? steps[activeStep].component : steps[totalSteps()-1].component;
+  let CurrentStep = activeStep >= steps.length ? steps[steps.length-1].component : steps[activeStep].component;
 
   return (
     <Box sx={{ width: '100%' }}>
-      <Stepper nonLinear activeStep={activeStep}>
-        {steps.map((step, index) => (
-          <Step key={step.label} completed={completed[index]}>
-            <StepButton color="inherit" onClick={handleStep(index)}>
-              {step.label}
-            </StepButton>
+      <Stepper activeStep={activeStep}>
+        {steps.map((step, index) => {
+          return (
+            <Step key={step.label}>
+              <StepLabel>{step.label}</StepLabel>
             </Step>
-        ))}
+          );
+        })}
       </Stepper>
-      <div>
-        {allStepsCompleted() ? (
+      {activeStep === steps.length ? (
         <React.Fragment>
           <ResultsStep {...props} />
           <Box sx={{ display: 'flex', flexDirection: 'row', pt: 2 }}>
@@ -104,25 +67,12 @@ export default function StepperWizard(props) {
               Back
             </Button>
             <Box sx={{ flex: '1 1 auto' }} />
-              <Button onClick={handleNext} sx={{ mr: 1 }}>
-                Next
+            <Button onClick={handleNext}>
+              {activeStep === steps.length - 1 ? 'Finish' : 'Next'}
             </Button>
-              {activeStep !== steps.length &&
-                (completed[activeStep] ? (
-                  <Typography variant="caption" sx={{ display: 'inline-block' }}>
-                    Step {activeStep + 1} already completed
-                  </Typography>
-                ) : (
-                  <Button onClick={handleComplete}>
-                    {completedSteps() === totalSteps() - 1
-                      ? 'Finish'
-                      : 'Complete Step'}
-                  </Button>
-                ))}
           </Box>
         </React.Fragment>
       )}
-      </div>
     </Box>
   );
 }