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

Expression RFC #52

Closed
wants to merge 1 commit into from
Closed

Expression RFC #52

wants to merge 1 commit into from

Conversation

mixonic
Copy link
Member

@mixonic mixonic commented May 17, 2015

View as HTML

This has been replaced by #53

@rlivsey
Copy link

rlivsey commented May 17, 2015

Huge 👍
A hook for cleaning up after ourselves would be useful but if destroy gets called at that point then that's fine.

@jamesarosen
Copy link

Will there be a way to register expressions that don't have a - in their name? For example, ember-can's can, ember-cli-i18n's t, and ember-i18n v4.0's t.

@tikotzky
Copy link

Whats the reasoning behind requiring a - in the expression name?
As I understand its required for components to align with web components. Is it just to help differentiate between helpers and expressions?

@mixonic
Copy link
Member Author

mixonic commented May 18, 2015

After discussion and consideration, I've decided to advocate for another direction. Instead of a wholly new type of template api, I'd like to introduce class-based helpers with a compute function having the same call signature as function helpers.

PR forthcoming, likely tomorrow morning at this point.

@mixonic mixonic closed this May 18, 2015
@mixonic
Copy link
Member Author

mixonic commented May 18, 2015

Regarding the dash- the new proposal will put the burden to change that requirement back on helpers themselves. With the glimmer refactor I think it is possible, will bring it up in discussion.

@mixonic
Copy link
Member Author

mixonic commented May 18, 2015

Supplanted by #53

@locks locks deleted the expression branch July 6, 2017 00:39
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.

6 participants