-
-
Notifications
You must be signed in to change notification settings - Fork 248
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
History back on step1 sucks #152
Comments
Fixed the issue, thanks for reporting @visualight |
techlab
added a commit
that referenced
this issue
Oct 3, 2022
Fix #152: History back on step1 not working
techlab
added a commit
that referenced
this issue
Oct 3, 2022
techlab
added a commit
that referenced
this issue
Oct 3, 2022
Fix #152: History back on step1 not working
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
Cannot history back() on step 1 when anchor navigation is enabled
To Reproduce
FIX
Edit jquery.smartWizard.js
Goto method : _setEvents
Locate the verification below
if(idx && _this3._isShowable(_this3.steps.eq(idx))) {...}
Replace with :
if(idx !== undefined && _this3._isShowable(_this3.steps.eq(idx))) {...}
In fact, idx return a zero (0) value for step 1 ... so you cannot simply verify idx as a boolean value (0 - null is false).
idx !== undefined // verify if value is defined
The text was updated successfully, but these errors were encountered: