Automattic / i18n-calypso

i18n JavaScript library on top of Jed originally used in Calypso
GNU General Public License v2.0
24 stars 19 forks source link

Handle options with new plural syntax #37

Closed yoavf closed 7 years ago

yoavf commented 7 years ago

This prevents the options object from being ignored with the "new" plural syntax. Example string:

this.translate(
                'Registering this domain for a company? + Add Organization Name',
                'Registering these domains for a company? + Add Organization Name',
                {
                    context: 'Domain contact information page',
                    comment: 'Count specifies the number of domain registrations',
                    count: 5
                }

Without this patch, the context and comment are ignored in the output.

Tug commented 7 years ago

LGTM 👍