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.

Sign up for GitHub

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

Jump to bottom

can.camelize #513

Closed
wclr opened this issue Oct 25, 2013 · 3 comments
Closed

can.camelize #513

wclr opened this issue Oct 25, 2013 · 3 comments
Milestone
2.0.1

Comments

@wclr
Copy link
Contributor

wclr commented Oct 25, 2013

I wonder why can.camelize was removed from can utils?
@justinbmeyer
Copy link
Contributor

It is not used anywhere.

Sent from my iPhone

On Oct 25, 2013, at 8:03 AM, Alex Osh notifications@github.com wrote:

I wonder why can.camelize was removed from can utils?

—
Reply to this email directly or view it on GitHub.

@wclr
Copy link
Contributor Author

wclr commented Oct 25, 2013

Maybe it could be used there #512

@imjoshdean
Copy link
Contributor

Yeah, #512 is a good reason to have it there (plus just because it wasn't used by the framework doesn't mean someone wasn't using it).

I'm going to add in a can.camelize back into can/util/string.js and as an added bonus you'll get the reverse action as well: can.hyphenate!

can.hyphenate('fooBarBaz'); // => foo-bar-baz

@ghost ghost assigned imjoshdean Oct 30, 2013
imjoshdean added a commit that referenced this issue Oct 30, 2013
@imjoshdean
Merge pull request #520 from bitovi/512-camelCaseComponent
1de0530 
Fixes #512 #513, sees the glorious return of can.camelize and his trusty sidekick, can.hyphenate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
2.0.1
Development

No branches or pull requests
3 participants
@justinbmeyer @imjoshdean @wclr