-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
allow to control active step when activeStepIndex change #25
Conversation
This PR allow you to control the active step outside the wizard.
|
Hello @mjangir any plan to merge these pull requests ? |
Pipeline failed. |
Sorry, I forgot to commit the import. Normally it should not failed now. |
@groomain Check line 18 if it is correct! |
src/useWizard.ts
Outdated
useEffect( | ||
() => { | ||
setCurrentStep(activeStepIndex); | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There should be a comma here. Did you find it working in your local machine? I'm sure it didn't.
is it good for you? |
The docs failed to be deployed. Is it because of the PR or juste a problem on github ? |
This PR allow you to control the active step outside the wizard.