fatihsolhan / v-onboarding

v-onboarding is a super-slim, fully-typed onboarding component for Vue 3
https://v-onboarding.fatihsolhan.com/
MIT License
164 stars 20 forks source link

exit / finish emits required #66

Closed DominikGanic closed 1 year ago

DominikGanic commented 1 year ago

Hi, there is an emit for "exit". But why not for "finish"?

As example: When I click the "X" button on top right of my onboarding, it executes the @exit=".." emit. But when the last step has been reached, there is a "Finish" button which executes as well the @exit=".." emit. But it should emit a @finish=".." function.

The reason for this is, I'm trying to seperate a cancel / finish function to save the state in my database. Now I have to write a complex condition to check if it's finished or has been exited manually by user. :/

Thanks!

fatihsolhan commented 1 year ago

Hey @DominikGanic, you are right. I have some other bug fixes as well, I'll add this to my list. They should be released this weekend

github-actions[bot] commented 1 year ago

:tada: This issue has been resolved in version 2.3.0 :tada:

The release is available on:

Your semantic-release bot :package::rocket:

DominikGanic commented 1 year ago

I've added a comment at this commit: https://github.com/fatihsolhan/v-onboarding/commit/ecb32258a5b826a66ceb6006438be9cbe1f2c12e

Looks like the following line there causes problems, since I've updated the package today.

index.2a3644af.js:1 DOMException: Failed to execute 'setAttribute' on 'Element': 'finish,' is not a valid attribute name.

image

fatihsolhan commented 1 year ago

oh, how did I miss that one.. fixing it now, thanks a lot @DominikGanic

DominikGanic commented 1 year ago

Thanks mate! Happens to the best ;)!