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

Update capitalize.js #61

Merged
merged 1 commit into from
Feb 1, 2019
Merged

Update capitalize.js #61

merged 1 commit into from
Feb 1, 2019

Conversation

marcelodolza
Copy link
Contributor

In this way, it also works for uppercase texts and all the following words, not just the first one.

In this way, it also works for uppercase texts and all the following words, not just the first one.
@freearhey
Copy link
Owner

But what if I need to "capitalize" only the first word?

@marcelodolza
Copy link
Contributor Author

Definition of "Capitalization" is the writing of a word with its first letter in uppercase and the remaining letters in lowercase. A word and not a sentence.
Just look at how this is handled in CSS.
But if you want to improve the code so that it works with "paragraphs" and "names". You can add parameters.

@freearhey
Copy link
Owner

Yeah, you right. I will try to do something about it.

@freearhey freearhey changed the base branch from master to dev February 1, 2019 11:39
@freearhey freearhey merged commit b53fbaf into freearhey:dev Feb 1, 2019
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.

2 participants