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

Document path based component/helper names #10243

Closed
sglanzer-deprecated opened this issue Jan 18, 2015 · 2 comments
Closed

Document path based component/helper names #10243

sglanzer-deprecated opened this issue Jan 18, 2015 · 2 comments

Comments

@sglanzer-deprecated
Copy link

Handlebars 2.0 allows path based helper names (e.g. my-component/sub). I started using this feature as soon as it was available to organize code (very nice organization with ember-cli and pods).

The ability to use this naming structure isn't documented for components anywhere that I can find. In addition, to use path based names and provide parameters the helper must be wrapped in [ ]. Like nested components, I found that answer in an issue thread handlebars-lang/handlebars.js#912.

I believe it would be helpful to document both the ability to nest component paths and the stipulations for using paths correctly.

As an example:

{{[sample-table/row] model=rowModel}}
{{[sample-table/cell] model=cellModel}}
@wagenet
Copy link
Member

wagenet commented Jan 21, 2015

@sglanzer We definitely should look into this. I know that not every Handlebars feature is actually supported in Ember, but it sounds like you've already used this without issue. However, we should verify that there aren't any unknown side-effects before we document this.

@rwjblue
Copy link
Member

rwjblue commented Aug 9, 2015

Nested helpers/components are definitely supported as of Ember 1.9.0 (when we upgraded to Handlebars 2.0.0).

We are still not quite sure what the API for nested angle bracket components, but will likely have some syntax specified (since it is an important issue).

Closing as this issue is somewhat old and isn't really describing a "bug" (but I would gladly accept a PR adding documentation).

@rwjblue rwjblue closed this as completed Aug 9, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants