Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

feat(ngBindOnceDirs.js): implementation of simple bindOnce directives #6815

Closed
georgebohnisch opened this issue Mar 23, 2014 · 2 comments
Closed

Comments

@georgebohnisch
Copy link

I have not seen or heard anything from the core team regarding the bindOnce issue, but I bet you guys are working on something, regarding the amount of +1s #5408 has received. But since I have not seen anything I thought someone would like this.

This is a super simple implementation which introduces many useful directives into the core with very little code. It has almost full parity with the popular bindonce library, in terms of the amount of directives it offers. It also works with promises, $timeouts, and any other values that are set after the initial digest.

I did not want to create all the Docs for each directive just yet. Wanted to get some feedback on whether or not this type of functionality would be merged in or not?

@caitp
Copy link
Contributor

caitp commented Mar 23, 2014

There is work being done on this, you can see discussion about it in the templating draft document https://docs.google.com/document/d/1f5VWROeTI2kJwVKbNsrHuEz5IqtZe14OpoxM9fEYJNU/ (I don't think expressionist.js or templating really prototype this yet, but in the common weeks we can probably have a shot at it and see what dogfooding it is like) ... Most likely, it will be possible to backport some version of whatever is decided on into Angular 1.x, it's not clear that the directive is the right approach.

So, this is being considered, absolutely. In the meantime, there are already third party modules which provide this functionality in the angular ecosystem, so hopefully there isn't too much of a rush for this. How does that sound to you?

@tbosch tbosch self-assigned this Mar 24, 2014
@tbosch
Copy link
Contributor

tbosch commented Mar 24, 2014

Hi @pixlgeek, we just started working on a "bind once" feature to be part of Angular 1.x.
Thanks for your PR though!

Closing this...

@tbosch tbosch closed this as completed Mar 24, 2014
@tbosch tbosch removed their assignment Mar 24, 2014
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants