Merge lp:~nikwen/ubuntu-terminal-app/ctrl-esc-keys-translatable into lp:ubuntu-terminal-app

Proposed by Niklas Wenzel
Status: Merged
Approved by: Alan Pope 🍺🐧🐱 🦄
Approved revision: 167
Merged at revision: 190
Proposed branch: lp:~nikwen/ubuntu-terminal-app/ctrl-esc-keys-translatable
Merge into: lp:ubuntu-terminal-app
Diff against target: 52 lines (+11/-5)
2 files modified
src/app/qml/Terminal.qml (+3/-1)
src/app/qml/extraPanel/CtrlKeys.qml (+8/-4)
To merge this branch: bzr merge lp:~nikwen/ubuntu-terminal-app/ctrl-esc-keys-translatable
Reviewer Review Type Date Requested Status
Ubuntu Phone Apps Jenkins Bot continuous-integration Approve
Ubuntu Terminal Developers Pending
Review via email: mp+238805@code.launchpad.net

Commit message

Make Ctrl and ESC keys translatable.

Description of the change

Make Ctrl and ESC keys translatable. For example, the Ctrl key is labeled "Strg" on German keyboards.

To post a comment you must log in.
167. By Niklas Wenzel

Add translators comments

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :

FAILED: Continuous integration, rev:167
No commit message was specified in the merge proposal. Click on the following link and set the commit message (if you want a jenkins rebuild you need to trigger it yourself):
https://code.launchpad.net/~nikwen/ubuntu-terminal-app/ctrl-esc-keys-translatable/+merge/238805/+edit-commit-message

http://91.189.93.70:8080/job/ubuntu-terminal-app-ci/122/
Executed test runs:
    SUCCESS: http://91.189.93.70:8080/job/generic-mediumtests-vivid/365
        deb: http://91.189.93.70:8080/job/generic-mediumtests-vivid/365/artifact/work/output/*zip*/output.zip
    SUCCESS: http://91.189.93.70:8080/job/ubuntu-terminal-app-vivid-amd64-ci/6

Click here to trigger a rebuild:
http://91.189.93.70:8080/job/ubuntu-terminal-app-ci/122/rebuild

review: Needs Fixing (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

The rebuild link somehow doesn't work for me. How can I make the Jenkins bot recognize my commit message?

Revision history for this message
Ubuntu Phone Apps Jenkins Bot (ubuntu-phone-apps-jenkins-bot) wrote :
review: Approve (continuous-integration)
Revision history for this message
Niklas Wenzel (nikwen) wrote :

Thanks for merging, even though this was for the pre-reboot version. :)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'src/app/qml/Terminal.qml'
2--- src/app/qml/Terminal.qml 2014-09-20 10:54:37 +0000
3+++ src/app/qml/Terminal.qml 2014-10-18 20:27:27 +0000
4@@ -185,7 +185,9 @@
5 visible: false
6 x: parent.x + (parent.width - width) / 2
7 y: parent.y + (kterm.height - height) / 2
8- circleButtonLable: "CTRL +"
9+
10+ // TRANSLATORS: "Ctrl" refers to the Ctrl key on the keyboard
11+ circleButtonLable: i18n.tr("CTRL +")
12 sideButtons: ["A", "C", "D", "S", "X", "Z"]
13
14
15
16=== modified file 'src/app/qml/extraPanel/CtrlKeys.qml'
17--- src/app/qml/extraPanel/CtrlKeys.qml 2014-09-20 10:54:37 +0000
18+++ src/app/qml/extraPanel/CtrlKeys.qml 2014-10-18 20:27:27 +0000
19@@ -22,25 +22,29 @@
20 spacing: units.gu(1)
21
22 Button {
23- Label {text: "ESC"; color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
24+ // TRANSLATORS: "ESC" refers to the ESC key on the keyboard
25+ Label {text: i18n.tr("ESC"); color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
26 width: btn_width
27 onClicked: keyPressed(Qt.Key_Escape, Qt.NoModifier)
28 }
29
30 Button {
31- Label {text: "Ctrl+Z"; color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
32+ // TRANSLATORS: "Ctrl" refers to the Ctrl key on the keyboard
33+ Label {text: i18n.tr("Ctrl+Z"); color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
34 width: btn_width*1.5
35 onClicked: keyPressed(Qt.Key_Z, Qt.ControlModifier)
36 }
37
38 Button {
39- Label {text: "Ctrl+X"; color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
40+ // TRANSLATORS: "Ctrl" refers to the Ctrl key on the keyboard
41+ Label {text: i18n.tr("Ctrl+X"); color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
42 width: btn_width*1.5
43 onClicked: keyPressed(Qt.Key_X, Qt.ControlModifier)
44 }
45
46 Button {
47- Label {text: "Ctrl+C"; color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
48+ // TRANSLATORS: "Ctrl" refers to the Ctrl key on the keyboard
49+ Label {text: i18n.tr("Ctrl+C"); color: buttonColor; anchors.centerIn: parent; fontSize: "medium"}
50 width: btn_width*1.5
51 onClicked: keyPressed(Qt.Key_C, Qt.ControlModifier)
52 }

Subscribers

People subscribed via source and target branches