angular-gettext icon indicating copy to clipboard operation
angular-gettext copied to clipboard

Directive doesn't need to be terminal anymore

Open gabegorelick opened this issue 10 years ago • 4 comments

See #132

gabegorelick avatar Dec 08 '14 18:12 gabegorelick

One thing I'm wondering about: does terminal: true prevent the body from being parsed?

And if we turn it off: will it be parsed / compiled?

In the directive, we do $animate.leave(oldContents); immediately, so if we can avoid Angular.JS from processing the contents of the directive, all the better.

But maybe this is covered by priority? I'm not intimate with the deep internals of the Angular.JS compile loop to properly know the true impact of this.

rubenv avatar Dec 08 '14 18:12 rubenv

This is why I didn't directly commit this patch ;)

I believe priority and terminal only relate to what happens when there are multiple directives on the same element. But I'm not an expert either.

gabegorelick avatar Dec 08 '14 18:12 gabegorelick

This is the best explanation I've seen: http://stackoverflow.com/a/15266968

gabegorelick avatar Dec 09 '14 17:12 gabegorelick

@rubenv @gabegorelick Is it going to to be merged? We've bumped into this as well.

princed avatar Jul 17 '15 07:07 princed