Is priority 900 in directive necessary?

[waldenraines]

I am having some problems with the 900 priority here: https://github.com/rubenv/angular-gettext/blob/master/src/directive.js#L4 and my other directives so I'm wondering if it is necessary. Everything seems to work fine if I remove it.

[rubenv]

Yes, it breaks the E2E test if you comment it out.

PhantomJS 1.9.1 (Mac OS X) Click Should not break ng-click FAILED
        expect element '#field' text toBe "Success"
        expected "Success" but was ""
PhantomJS 1.9.1 (Mac OS X) Click Should not break ng-click for translated strings FAILED
        expect element '#field' text toBe "Success"
        expected "Success" but was ""
PhantomJS 1.9.1 (Mac OS X): Executed 2 of 2 (2 FAILED) ERROR (0.318 secs / 0.234 secs)

I'm not entirely happy with the way the transclusion is currently handled, but it does work for my test cases.

Do you have a test case to demonstrate the problem you're seeing? Would be nice to have it in the test suite.

[waldenraines]

The problem between directives was actually on my end so no test case, sorry for the false alarm.

I'll leave it up to you whether to close this issue or leave it open.

[rubenv]

It needs a high priority so that it registers before ngClick, unless there's a good other way to do this, we'll need it.