-
Notifications
You must be signed in to change notification settings - Fork 7.5k
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
Components are now accessible via camelCase
and UpperCamelCase
me…
#3439
Components are now accessible via camelCase
and UpperCamelCase
me…
#3439
Conversation
@@ -361,7 +361,7 @@ class Component { | |||
|
|||
// If no componentClass in options, assume componentClass is the name lowercased |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
comment might need to be updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this comment could just be removed?
de10475
to
ff54edc
Compare
Looks like this needs to be rebased against master |
ff54edc
to
78db9f2
Compare
@gkatsev I temporarily removed |
…ans. - addChild - getChild - getComponent - registerComponent
78db9f2
to
f94c3d9
Compare
@gkatsev updated again. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Basically, components will be stored internally via the toTitleCase
name?
LGTM.
Test failing because of the firefox issue.
@gkatsev, Is that a known issue? I wasn't having that issue with FF 48, but I am having it with 49.0.1, does that have to do with me? (removing the extraneous comment) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM and works for me.
@hartman unfortunately, it doesn't. |
Fixes #3436.
Description
Components are not uniformly accessible through the same names.
Example:
getChild
cannot retrieve a component by acamelCase
nameaddChild
can retrieve, then add a component by acamelCase
nameSpecific Changes proposed
Enable components to be accessed via both
camelCase
andUpperCamelCase
means.Requirements Checklist
…ans.