Merge pull request #2352 from matrix-org/travis/fix-notification-bell

Fix translation error on notification icon
matrix-org · Dec 13, 2018 · bae9996 · bae9996
2 parents 366f343 + 49769a4
commit bae9996
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/src/components/structures/RightPanel.js b/src/components/structures/RightPanel.js
@@ -326,7 +326,8 @@ module.exports = React.createClass({
             let notifCount = 0;
             MatrixClientPeg.get().getRooms().forEach(r => notifCount += (r.getUnreadNotificationCount('highlight') || 0));
             if (notifCount > 0) {
-                notifCountBadge = <div title={_t("%counts Notifications")}>{ formatCount(notifCount) }</div>;
+                const title = _t("%(count)s Notifications", {count: formatCount(notifCount)});
+                notifCountBadge = <div title={title}>{ formatCount(notifCount) }</div>;
             }
 
             headerButtons = [

diff --git a/src/i18n/strings/en_EN.json b/src/i18n/strings/en_EN.json
@@ -1144,6 +1144,8 @@
     "%(count)s Members|other": "%(count)s Members",
     "%(count)s Members|one": "%(count)s Member",
     "Invite to this room": "Invite to this room",
+    "%(count)s Notifications|other": "%(count)s Notifications",
+    "%(count)s Notifications|one": "%(count)s Notification",
     "Files": "Files",
     "Notifications": "Notifications",
     "Hide panel": "Hide panel",