Skip to content
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

onStop event is not emitted #8

Closed
mc-mosa opened this issue Apr 29, 2019 · 7 comments
Closed

onStop event is not emitted #8

mc-mosa opened this issue Apr 29, 2019 · 7 comments
Labels
bug Something isn't working

Comments

@mc-mosa
Copy link
Contributor

mc-mosa commented Apr 29, 2019

The onStop event is not emitted when the countdown is stopped

@bi6o
Copy link

bi6o commented May 9, 2019

I am using this library on a project and I noticed that calling the stopCountdown() method stops the countdown. When I start the countdown again, the time passed is taken into account

This is observable also in the demo page https://vac.js.org/examples/methods/all-methods.html.
Just click on stop, wait for some time, and click start again.

I could just not render the component as a work around, e.g. v-if="isCounting" instead of using the stopCountdown() method

Is the issue I mentioned related to this event not being emitted? And is it expected for the fix to be merged?

@mc-mosa
Copy link
Contributor Author

mc-mosa commented May 10, 2019

No this doesn't have anything to do with the problem you mentioned.

I have noticed it as well but don't have much time to look into it at the moment. I don't know if that's an intended behavior or just a bug. Will try to look into it as soon as I have the time.

@bi6o
Copy link

bi6o commented May 10, 2019

Doesn't seem to me like an intended behavior. Thank you for that, I will keep my 👀 open for this one

@mlinquan mlinquan added the bug Something isn't working label May 20, 2019
@mlinquan
Copy link
Owner

Merged

@bi6o
Copy link

bi6o commented May 20, 2019

Hey, I saw that you merged a fix this morning. I have updated the library and the bug is still happening, that's also true for the live example that I mentioned before https://vac.js.org/examples/methods/all-methods.html . Am I missing something?

@mc-mosa
Copy link
Contributor Author

mc-mosa commented May 22, 2019

Hey, sorry.. the merge was for the bug mentioned in the title of this issue. I haven't got the chance to look into the bug we discussed earlier, will try to do that this weekend.

@mlinquan
Copy link
Owner

@bi6o @mc-mosa The issue was fixed in version 1.0.14.

@mc-mosa mc-mosa closed this as completed May 28, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants