Skip to content
This repository has been archived by the owner on Feb 17, 2021. It is now read-only.

No event called after tour is finished #188

Closed
Tombre opened this issue Jun 22, 2015 · 3 comments · Fixed by #194
Closed

No event called after tour is finished #188

Tombre opened this issue Jun 22, 2015 · 3 comments · Fixed by #194
Assignees
Labels
Milestone

Comments

@Tombre
Copy link

Tombre commented Jun 22, 2015

Hi guys, not sure this is a bug or not, but I've noticed that when the skipIfNoElement option is set to false, the onEnd callback is not called once the final step in a tour is completed (the user clicks "done").

The issue looks like it comes from the goToStepWithTarget method, specifically line 1671:

else if (currStepNum + direction >= 0 && currStepNum + direction < currTour.steps.length) {

Since by the end of a tour, the tour length is the same length as currStepNum + direction, changeStepCb is never run and thus the onEnd callbacks are neither.

Is this expected behavior?

@kate2753 kate2753 modified the milestones: 0.3.0, 0.2.5 Jun 23, 2015
@kate2753
Copy link
Contributor

@Tombre, thank you for the issue report! This does appear to be a bug. I'm adding it to the 0.2.5 milestone, so we can get a fix out for you.

@kate2753 kate2753 self-assigned this Jun 23, 2015
@Tombre
Copy link
Author

Tombre commented Jun 24, 2015

Thanks Kate!

kate2753 added a commit to kate2753/hopscotch that referenced this issue Jun 25, 2015
kate2753 added a commit to kate2753/hopscotch that referenced this issue Jun 25, 2015
kate2753 added a commit that referenced this issue Jul 20, 2015
#188: No event called after tour is finished
@Ettore495
Copy link

FIx this please :(

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants