Closed Bug 719402 Opened 12 years ago Closed 11 years ago

Implement plural forms for real in l10n module.


(Add-on SDK Graveyard :: General, defect, P1)



(Not tracked)



(Reporter: ochameau, Assigned: ochameau)



(1 file)

Currently, l10n module only implements engligh plural forms:

We need to implement specific set of forms for each language, by following rules described here:

We may use code provided by PluralForms.jsm:
Attached file Pull request 332
-> more information in the pull request.
Assignee: nobody → poirot.alex
Here is the documentation for current XUL way of doing localization with plural forms:
Comment on attachment 591814 [details]
Pull request 332

As you already reviewed the very first l10n patch, I'd think that you may review this one too? Feel free to redirect the review to Irakli if you do not feel reviewing JS code.
Attachment #591814 - Flags: review?(warner-bugzilla)
I've added comments to the pull request.. looks mostly good, one potential real bug, a handful of style recommendations.
Comment on attachment 591814 [details]
Pull request 332

Looks good: r+ with or without the remaining minor suggestions on the pull-request
Attachment #591814 - Flags: review?(warner-bugzilla) → review+
Commit pushed to master at
Merge pull request #332 from ochameau/implement-plural-forms

Bug 719402 - Implement plural forms r=@warner
Closed: 11 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.