Skip to content
This repository was archived by the owner on Jun 23, 2026. It is now read-only.

Add reset method#46

Merged
siddii merged 3 commits into
siddii:masterfrom
59023g:master
Dec 15, 2014
Merged

Add reset method#46
siddii merged 3 commits into
siddii:masterfrom
59023g:master

Conversation

@59023g

@59023g 59023g commented Feb 10, 2014

Copy link
Copy Markdown
Contributor

Combined start and stop

@59023g

59023g commented Feb 10, 2014

Copy link
Copy Markdown
Contributor Author

Great directive. Thank you. I'm new to this, so please don't hesitate to right a wrong.

On countdown, I was looking for a reset function.

PR above. Thoughts?

@siddii

siddii commented Feb 10, 2014

Copy link
Copy Markdown
Owner

I am looking at this PR. Is the use case is to reset the running countdown timer?

@59023g

59023g commented Feb 10, 2014

Copy link
Copy Markdown
Contributor Author

Yes

Added semicolon
@siddii

siddii commented Feb 10, 2014

Copy link
Copy Markdown
Owner

Can you please include a test case for this? Preferably an html file under examples folder.
I have a feeling the countdown stops when that method executes because of $scope.clear() being the last statement that its calling.

@siddii siddii closed this Mar 18, 2014
@siddii siddii reopened this Apr 18, 2014
@siddii siddii mentioned this pull request Apr 18, 2014
@brayann

brayann commented Dec 15, 2014

Copy link
Copy Markdown
Contributor

@siddii are you merging this anytime soon?

@siddii

siddii commented Dec 15, 2014

Copy link
Copy Markdown
Owner

OK, somehow this got missed. I am merging now. Let me know if everything looks fine after the merge.

siddii added a commit that referenced this pull request Dec 15, 2014
@siddii siddii merged commit 16bef4b into siddii:master Dec 15, 2014
@siddii

siddii commented Dec 16, 2014

Copy link
Copy Markdown
Owner

@brayann

brayann commented Dec 16, 2014

Copy link
Copy Markdown
Contributor

Thank you !

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants