toTitleCase is removing full stops .

[dan-scrumsdotcom]

console.log(toTitleCase('hello there old friend...'))

Maybe intentional? Not sure :)

[dan-scrumsdotcom]

Same log with console.log(toTitleCase('hello. there. old. friend...'))

[gustavoguichard]

Hi @danharten-sovtech , thanks for reporting.

This is actually how lodash works as well:

The goal of this library is not to be equivalent to lodash but in this case it is.
The casing functions all have the same foundation which is the words function that detects words and joins them with a different casing. If you think about toTitleCase as being from the same family as toCamelCase it becomes clearer:

toCamelCase('hello there old friend...')
// expected to be:
'helloThereOldFriend'

// this would be weird
'helloThereOldFriend...'

But as I think about it I realized we should probably have a different naming convention for toLowerCase and toUpperCase that have a different foundation therefore will keep the dots.

@jly36963 , @p9f , what do you guys think about it?

[jly36963]

I would expect all punctuation to be removed
Eg:

toTitleCase('my-kebab-case') // 'My Kebab Case'
toTitleCase('my_snake_case') // 'My Snake Case'
toTitleCase('my.path.case') // 'My Path Case'

However, I definitely think we could include specific examples in the documentation to make this clear. As a user, I could see myself being surprised by this behavior as well.

[jly36963]

After trying a few different inputs, I do see this though:

toTitleCase('my.path.case[0]');  // "My Path Case [ 0 ]"

/*
// lodash example
_.startCase('my.path.case[0]') // "My Path Case 0"
*/

In this case, I would also expect [ and ] to be removed (as lodash does).

[gustavoguichard]

/*
// lodash example
_.startCase('my.path.case[0]') // "My Path Case 0"
*/

Even though the goal is not to have the same behavior as lodash I agree we should fix this... could you create another issue for it?

As for this issue, let's make it clearer in the docs. Let's check out what lodash does since we are talking about it:

## lowerCase
Converts string, as space separated words, to lower case.

## toLower
Converts string, as a whole, to lower case just like [String#toLowerCase](https://mdn.io/toLowerCase).

I thing we should name the methods differently for v2, and implement lowerCase and upperCase, like they do... for now we can have a different section in the README

[gustavoguichard]

@jly36963 maybe we should add '\' | '|' | '(' | ')' | '[' | ']' | '{' | '}' to separators and also here.

That should make our library a lil more similar to what people are used to.

[dan-scrumsdotcom]

Wow, clocked off for the day to come back to a totally explained and resolved issue. Thank you @gustavoguichard @jly36963!