Support context-extraction from filters.

[rubenv]

See rubenv/angular-gettext#103

Pull requests welcome

{{"1 car" | translate:"context"}}

[gabegorelick]

Is there any other parameter you'd want to pass as the first argument to the filter? My only concern is that filter parameters don't really scale to large permutations of arguments, and there's a lot of ways you can traditionally call gettext (context, plurals, etc.)

[rubenv]

You're right: filters don't scale well.

I don't like them, but we need them for some cases (e.g. translated attributes).

See #39: we'll need a second filter to do plurals.

[gabegorelick]

Can you pass object literals to filters? Something like translate:{context: ""}? It's future-proof and still extractable.

[rubenv]

Yup, that could work as well. I'd be cool with that (it means we only need one filter!).

We'd need to change the filter implementation as well then.

[gabegorelick]

It's also not too late to remove the {{'' | translate:'context' }} syntax if we decide object literals is better.

[gabegorelick]

angular-translate uses object literal-strings: https://github.com/angular-translate/angular-translate/wiki/Dynamize-your-translations#parametrizing-the-translate-filter

We probably have to remove support for strings because angular doesn't like object literals passed to filters.

[rubenv]

@gabegorelick I'm considering a release of angular-gettext without context support in filters. This gives us an opportunity to get the fixes out there and gives us more time to figure out how to best move forward with filters.

Are you OK with that?

[gabegorelick]

Fine by me.

On Tuesday, December 16, 2014, Ruben Vermeersch notifications@github.com
wrote:

@gabegorelick https://github.com/gabegorelick I'm considering a release
of angular-gettext without context support in filters. This gives us an
opportunity to get the fixes out there and gives us more time to figure out
how to best move forward with filters.

Are you OK with that?

—
Reply to this email directly or view it on GitHub
#63 (comment)
.

[rubenv]

Moving to 2.1.