Merge lp:~pitti/messaging-app/i18n-fixes into lp:messaging-app

Proposed by Martin Pitt
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 200
Merged at revision: 205
Proposed branch: lp:~pitti/messaging-app/i18n-fixes
Merge into: lp:messaging-app
Diff against target: 50 lines (+6/-6)
3 files modified
src/qml/MainPage.qml (+1/-1)
src/qml/Messages.qml (+1/-1)
src/qml/ThreadDelegate.qml (+4/-4)
To merge this branch: bzr merge lp:~pitti/messaging-app/i18n-fixes
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+232507@code.launchpad.net

Commit message

Do not mark single space for translation, and fix wrong macros in plural forms of "Attachments" strings.

Description of the change

Fix i18n issues, see commit logs.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)
Revision history for this message
Gustavo Pichorim Boiko (boiko) wrote :

Looks good!

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/qml/MainPage.qml'
2--- src/qml/MainPage.qml 2014-08-12 15:14:25 +0000
3+++ src/qml/MainPage.qml 2014-08-28 05:36:01 +0000
4@@ -34,7 +34,7 @@
5 }
6
7 state: selectionMode ? "select" : searching ? "search" : "default"
8- title: selectionMode ? i18n.tr(" ") : i18n.tr("Chats")
9+ title: selectionMode ? " " : i18n.tr("Chats")
10
11 bottomEdgeEnabled: !selectionMode && !searching
12 bottomEdgePageComponent: Messages {
13
14=== modified file 'src/qml/Messages.qml'
15--- src/qml/Messages.qml 2014-08-20 15:34:00 +0000
16+++ src/qml/Messages.qml 2014-08-28 05:36:01 +0000
17@@ -113,7 +113,7 @@
18
19 title: {
20 if (selectionMode) {
21- return i18n.tr(" ")
22+ return " "
23 }
24
25 if (landscape) {
26
27=== modified file 'src/qml/ThreadDelegate.qml'
28--- src/qml/ThreadDelegate.qml 2014-08-12 15:14:25 +0000
29+++ src/qml/ThreadDelegate.qml 2014-08-28 05:36:01 +0000
30@@ -65,16 +65,16 @@
31 attachmentCount = imageCount + videoCount + contactCount
32
33 if (imageCount > 0 && attachmentCount == imageCount) {
34- return i18n.tr("Attachment: %1 image", "Attachments: %s images").arg(imageCount)
35+ return i18n.tr("Attachment: %1 image", "Attachments: %1 images").arg(imageCount)
36 }
37 if (videoCount > 0 && attachmentCount == videoCount) {
38- return i18n.tr("Attachment: %1 video", "Attachments: %s videos").arg(videoCount)
39+ return i18n.tr("Attachment: %1 video", "Attachments: %1 videos").arg(videoCount)
40 }
41 if (contactCount > 0 && attachmentCount == contactCount) {
42- return i18n.tr("Attachment: %1 contact", "Attachments: %s contacts").arg(contactCount)
43+ return i18n.tr("Attachment: %1 contact", "Attachments: %1 contacts").arg(contactCount)
44 }
45 if (attachmentCount > 0) {
46- return i18n.tr("Attachment: %1 file", "Attachments: %s files").arg(attachmentCount)
47+ return i18n.tr("Attachment: %1 file", "Attachments: %1 files").arg(attachmentCount)
48 }
49 return eventTextMessage
50 }

Subscribers

People subscribed via source and target branches