NOTE: Improve i18n pluralization

7 years ago · 8a8972f7ac
parent a76b1389e6
commit 8a8972f7ac
1 changed files with 4 additions and 0 deletions
--- a/js/notifications.js
+++ b/js/notifications.js
@ -64,6 +64,10 @@
            var message;
            var iconUrl;

+            // NOTE: i18n has more complex rules for pluralization than just
+            // distinguishing between zero (0) and other (non-zero),
+            // e.g. Russian:
+            // http://docs.translatehouse.org/projects/localization-guide/en/latest/l10n/pluralforms.html
            var newMessageCount = [
                this.length,
                this.length === 1 ? i18n('newMessage') : i18n('newMessages')