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

Fix word count in ember. #3033

Merged
merged 1 commit into from
Jun 23, 2014
Merged

Fix word count in ember. #3033

merged 1 commit into from
Jun 23, 2014

Conversation

halfdan
Copy link
Contributor

@halfdan halfdan commented Jun 22, 2014

The word count in ember is currently incorrect as it ignored new lines.

When entering the following text the word count shows 3 instead of 4:

foo bar
baz rasper

No issue. Sorry @ErisDS ✌️

@darvelo
Copy link
Contributor

darvelo commented Jun 22, 2014

@halfdan I changed the handlebars helper to handle 0 words a while back. Does that also need to be fixed in light of this change?

@halfdan
Copy link
Contributor Author

halfdan commented Jun 22, 2014

@darvelo Nope, that looks good. I discovered a bug in my fix though which I have just fixed.

@darvelo
Copy link
Contributor

darvelo commented Jun 22, 2014

👍

@ErisDS
Copy link
Member

ErisDS commented Jun 23, 2014

👍 keep em coming

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

Successfully merging this pull request may close these issues.

3 participants