Skip to content

"count-to" needs to be changed to "countTo" #3

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

Open
onearmfrog opened this issue Oct 15, 2013 · 9 comments
Open

"count-to" needs to be changed to "countTo" #3

onearmfrog opened this issue Oct 15, 2013 · 9 comments

Comments

@onearmfrog
Copy link

I believe the instructions for implementing the directives need to be changed -
"count-to" doesn't work, the right namespace is "countTo".

@wholypantalones
Copy link

The documentation needs to be updated, I can't even get this to work.

@sparkalow
Copy link
Owner

The directive used on the demo is "count-to" and works fine.

@wholypantalones this has not been tested with angular after version 1.0.7. I'll note this in the readme while updating for compatibility.

@seriema
Copy link

seriema commented Jun 21, 2014

Your source code says count-to:

// angular-count-to / src / count-to.js
var countTo = angular.module('countTo', [])

@jkelleher
Copy link

Agree with @seriema - dependency should be 'countTo' and not 'count-to' as directed in docs.

@aendra-rininsland
Copy link

👍 Yeah, just scratched my head for about 10 minutes figuring that out.

@seriema
Copy link

seriema commented Jul 30, 2015

There seems to be a PR #11 to update the docs at least. I think the code should change, but at least the readme would be a safe (non-breaking) update.

@dannyblv
Copy link

👍 yep, the documentation needs to be updated ..

@birkmann
Copy link

birkmann commented Feb 5, 2016

countTo ;)

@peteroid
Copy link

+1

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

No branches or pull requests

9 participants