Merge lp:~seb128/ubuntu-system-settings/security-translation-context into lp:ubuntu-system-settings

Proposed by Sebastien Bacher
Status: Merged
Approved by: Ken VanDine
Approved revision: 982
Merged at revision: 989
Proposed branch: lp:~seb128/ubuntu-system-settings/security-translation-context
Merge into: lp:ubuntu-system-settings
Diff against target: 25 lines (+2/-2)
2 files modified
plugins/security-privacy/PhoneLocking.qml (+1/-1)
po/CMakeLists.txt (+1/-1)
To merge this branch: bzr merge lp:~seb128/ubuntu-system-settings/security-translation-context
Reviewer Review Type Date Requested Status
Ken VanDine Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+233016@code.launchpad.net

Commit message

[security] give a translation context to the 'None' string

Description of the change

[security] give a translation context to the 'None' string

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
Ken VanDine (ken-vandine) wrote :

Wouldn't the string "Swipe" make more sense than "None"? It's commonly referred to as "Swipe to unlock", I'd prefer to change the string, but I know it's pretty late for that. thoughts?

Revision history for this message
Sebastien Bacher (seb128) wrote :

That's what the design recommends, better to check with mpt I guess

Revision history for this message
Ken VanDine (ken-vandine) wrote :

I'll +1 the change

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'plugins/security-privacy/PhoneLocking.qml'
2--- plugins/security-privacy/PhoneLocking.qml 2014-08-28 18:11:36 +0000
3+++ plugins/security-privacy/PhoneLocking.qml 2014-09-02 09:49:38 +0000
4@@ -42,7 +42,7 @@
5 anchors.right: parent.right
6
7 ListItem.SingleValue {
8- property string swipe: i18n.tr("None")
9+ property string swipe: qsTr("None", "Unlock with swipe")
10 property string passcode: i18n.tr("PIN code")
11 property string passphrase: i18n.tr("Passphrase")
12
13
14=== modified file 'po/CMakeLists.txt'
15--- po/CMakeLists.txt 2014-08-26 09:42:08 +0000
16+++ po/CMakeLists.txt 2014-09-02 09:49:38 +0000
17@@ -28,7 +28,7 @@
18 --copyright=\"Canonical Ltd.\"
19 --package-name ubuntu-system-settings
20 --qt --c++ --add-comments=TRANSLATORS
21- --keyword=QT_TR_NOOP
22+ --keyword=QT_TR_NOOP --keyword=qsTr:1,2c
23 --keyword=tr --keyword=tr:1,2 --from-code=UTF-8
24 ${QMLFILES} "${CMAKE_CURRENT_BINARY_DIR}/settings.js"
25 WORKING_DIRECTORY "${CMAKE_CURRENT_SOURCE_DIR}"

Subscribers

People subscribed via source and target branches