Merge lp:~seb128/dialer-app/tweak-some-strings into lp:dialer-app

Proposed by Sebastien Bacher
Status: Merged
Approved by: Gustavo Pichorim Boiko
Approved revision: 436
Merged at revision: 440
Proposed branch: lp:~seb128/dialer-app/tweak-some-strings
Merge into: lp:dialer-app
Diff against target: 21 lines (+2/-4)
1 file modified
src/qml/DialerPage/DialerPage.qml (+2/-4)
To merge this branch: bzr merge lp:~seb128/dialer-app/tweak-some-strings
Reviewer Review Type Date Requested Status
Gustavo Pichorim Boiko (community) Approve
PS Jenkins bot continuous-integration Approve
Review via email: mp+265096@code.launchpad.net

Commit message

Use "Phone" rather than "Keypad" for the default title, as recommended by
design.
Change "Flight Mode" to use proper casing

Description of the change

Use "Phone" rather than "Keypad" for the default title, as recommended by
design.
Change "Flight Mode" to use proper casing

To post a comment you must log in.
436. By Sebastien Bacher

Use "Phone" rather than "Keypad" for the default title, as recommended by
design.
Change "Flight Mode" to use proper casing

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
=== modified file 'src/qml/DialerPage/DialerPage.qml'
--- src/qml/DialerPage/DialerPage.qml 2015-06-18 20:01:19 +0000
+++ src/qml/DialerPage/DialerPage.qml 2015-07-17 09:01:55 +0000
@@ -74,15 +74,13 @@
74 return " "74 return " "
75 }75 }
76 } else if (telepathyHelper.flightMode) {76 } else if (telepathyHelper.flightMode) {
77 return i18n.tr("Flight mode")77 return i18n.tr("Flight Mode")
78 } else if (mainView.account && mainView.account.simLocked) {78 } else if (mainView.account && mainView.account.simLocked) {
79 return i18n.tr("SIM Locked")79 return i18n.tr("SIM Locked")
80 } else if (mainView.account && mainView.account.networkName != "") {80 } else if (mainView.account && mainView.account.networkName != "") {
81 return mainView.account.networkName81 return mainView.account.networkName
82 } else if (multipleAccounts && !mainView.account) {82 } else if (multipleAccounts && !mainView.account) {
83 // TODO: check what should be displayed when there are multiple accounts83 return i18n.tr("Phone")
84 // but no default selected
85 return i18n.tr("Keypad")
86 }84 }
87 return i18n.tr("No network")85 return i18n.tr("No network")
88 }86 }

Subscribers

People subscribed via source and target branches