Merge lp:~ken-vandine/ubuntu-system-settings/rtm-14.09_rotation_lock_string into lp:ubuntu-system-settings/rtm-14.09

[Ken VanDine]

[orientation] String change to Rotation Lock

[orientation] String change to Rotation Lock

[Sebastien Bacher (seb128)]

Thanks but we said we wouldn't change strings in rtm anymore no?

review: Disapprove

[Ken VanDine (ken-vandine)]

Yes, we did say that. The rotation indicator just landed in the rtm images which uses the string "Rotation Lock", not matching settings. The design spec changed the wording for settings a while ago, but no bug was filed. The indicator isn't translated yet at all, so we need to decide if we want to keep settings as is having a string that has been translated which doesn't match the indicator. I'm more concerned about the new string introduced by the indicator, which isn't translated. That string shows whenever you pull down the indicators. In settings we only see it when you launch settings. I'd actually vote to change the string in the indicator so we can benefit from the already translated string for now.

[Sebastien Bacher (seb128)]

there are more chance than somebody start the settings than use the indicator, especially than the indicator is not displaying an icon by default (it's in the list when you pulldown but it's less discoverable this way)

[lp:~ken-vandine/ubuntu-system-settings/rtm-14.09_rotation_lock_string]

933. By Ken VanDine on 2014-11-18

merged from parent

[Sebastien Bacher (seb128)]

Ok, we need to organize a langpack update after that landing/before image promotion though, otherwise the string is going to show untranslated for users

review: Approve

[PS Jenkins bot (ps-jenkins)]

PASSED: Continuous integration, rev:933
http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-ci/16/
Executed test runs:
    SUCCESS: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/17
        deb: http://jenkins.qa.ubuntu.com/job/ubuntu-system-settings-rtm-14.09-14.09-armhf-ci/17/artifact/work/output/*zip*/output.zip

Click here to trigger a rebuild:
http://s-jenkins.ubuntu-ci:8080/job/ubuntu-system-settings-rtm-14.09-ci/16/rebuild

review: Approve (continuous-integration)

=== modified file 'plugins/orientation-lock/orientation-lock.settings'
--- plugins/orientation-lock/orientation-lock.settings	2014-09-24 17:10:03 +0000
+++ plugins/orientation-lock/orientation-lock.settings	2014-11-18 17:29:19 +0000
@@ -1,6 +1,6 @@
 {
     "icon": "orientation-lock",
-    "name": "Orientation Lock",
+    "name": "Rotation Lock",
     "translations": "ubuntu-system-settings",
     "category": "uncategorized-top",
     "priority": 0,