-
Notifications
You must be signed in to change notification settings - Fork 24
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
Hi Guys!, there is way to go the previous item? #55
Comments
I haven't planned for this feature. You could restart the onboarding if the user have missed something or needs a reminder. |
I did have an end user that requested this feature. It's not a deal breaker, but it might be a nice to have in the stepBuilder if not to hard to implement. |
It's hard to implement. The whole logic around the stepper is removing an item from the beginning of the list on next step. If I have removed the item from the list, I can't go back to it. I haven't revisited this logic since the first publish. Feel free to look around and suggest a PR with changes. |
Riiight. I see it. Instead of explicitly setting the stepIndexes when showing just a subset of the onboarding steps, one would need to set something like a startAt and endAt index value. Then you wouldn't need to remove the index as you went but could just increase or decrease the currentIndex until it is above endAt or below startAt... Something like that. I might try to do a PR if I have time. Although the downside of this would be that you would have to run a sequential set of indexes (as now you can pick and choose which ones to show with the explicit list. |
Right, and I have made it support an arbitrary list of indexes not necessarily in order. See showWithIndexes |
In that case, it might be possible to have a totalIndex list and a
selectedIndex list. When the onboarding is shown, the selectedIndex list
gets populated with either the indexes passed or the indexes of all of the
steps. Then the current index points to the selectedIndex list, which is
used to look up the step from the totalIndex list... A bit clunky maybe but
that would be where I would start tinkering.
+420 608 577 012
…On Sat, Dec 16, 2023 at 3:16 PM Zlati Pehlivanov ***@***.***> wrote:
Right, and I have made it support an arbitrary list of indexes not
necessarily in order. See showWithIndexes
—
Reply to this email directly, view it on GitHub
<#55 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAETNITQNO7ZKGDYL276MELYJWULLAVCNFSM6AAAAAA6MISAO6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNJYHAZDSMRQHE>
.
You are receiving this because you commented.Message ID:
***@***.***>
|
No description provided.
The text was updated successfully, but these errors were encountered: