1189680 - Restricted Profiles: Review new strings for profile settings UI

Status: VERIFIED FIXED

(Firefox for Android Graveyard :: Profile Handling, defect)

Description

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: restrictions-setup.png

Since bug 965151 landed we allow the device admin (e.g. parent) to configure the restrictions that are enforced for a restricted profile (e.g. child). This is part of the restricted profile setup UI of the Android system. See attached screenshot how this looks like.

Every restriction can have a title and a description. The strings I added are monotonic and the description is often not very helpful or the same as the title.

These are our current strings:

> <!ENTITY restriction_disallow_import_settings_title "Disallow importing settings">
> <!ENTITY restriction_disallow_import_settings_description "Do not allow to import settings from other system browsers.">

> <!ENTITY restriction_disallow_addons_title "Disallow add-ons">
> <!ENTITY restriction_disallow_addons_description "Disallow installation of add-ons.">

> <!ENTITY restriction_disallow_devtools_title "Disallow developer tools">
> <!ENTITY restriction_disallow_devtools_description "Disallow usage of developer tools.">

> <!ENTITY restriction_disallow_customize_home_title "Disallow customizing home">
> <!ENTITY restriction_disallow_customize_home_description "Disallow customizing home panels.">

> <!ENTITY restriction_disallow_private_browsing_title "Disallow private browsing">
> <!ENTITY restriction_disallow_private_browsing_description "Disallow private browsing mode.">

> <!ENTITY restriction_disallow_location_services_title "Disallow location services">
> <!ENTITY restriction_disallow_location_services_description "Disallow sharing of location data to improve geolocation service.">

> <!ENTITY restriction_disallow_display_settings_title "Disallow display settings">
> <!ENTITY restriction_disallow_display_settings_description "Disallow changing of advanced display settings.">

> <!ENTITY restriction_disallow_clear_history_title "Disallow clearing history">
> <!ENTITY restriction_disallow_clear_history_description "Disallow clearing of browser history.">

> <!ENTITY restriction_disallow_master_password_title "Disallow master password">
> <!ENTITY restriction_disallow_master_password_description "Disallow setting a master password for logins.">


These three are still in there but are very likely going away in bug 1189414

> <!ENTITY restriction_disallow_tools_menu_title "Disallow Tools menu">
> <!ENTITY restriction_disallow_tools_menu_description "Hide Tools menu from UI.">

> <!ENTITY restriction_disallow_report_site_issue_title "Disallow \'Report site issue\'">
> <!ENTITY restriction_disallow_report_site_issue_description "Hide \'Report site issue\' menu item.">

> <!ENTITY restriction_disallow_apps_title "Disallow apps">
> <!ENTITY restriction_disallow_apps_description "Disallow installing apps from Firefox Marketplace.">

Comment 1

[Sebastian Kaspari (:sebastian; :pocmo)]

Robin, Anthony: Can you help me to come up with strings that are actually helpful and can be understood by a non-developer?

Comment 2

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

As a general pass, I'd use "disable" instead of "disallow".

Also, "Report Site Issue" is only available on Nightly builds I think? Do we need that item called out here?

Maybe something like:

Disable add-ons installation

Disable apps installation

Hide tools menu

Hide import settings

Have you guys thought about grouping these options? "disables" and the "hides" maybe? it might make for easier messaging as well since some of these are obviously features that users want to get "just out of reach" rather than "completely hidden"

Comment 3

[Sebastian Kaspari (:sebastian; :pocmo)]

As collaborating in bugzilla isn't that great I created the following etherpad for this:
https://mobile.etherpad.mozilla.org/97

Comment 4

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to Anthony Lam (:antlam) from comment #2)
> As a general pass, I'd use "disable" instead of "disallow".

Alright. I'll try to update the strings accordingly.

> Also, "Report Site Issue" is only available on Nightly builds I think? Do we
> need that item called out here?

Yeah, this one is going away. I updated the etherpad with the latest version.

Comment 5

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Awesome, thanks Sebastian!

I left some comments and questions in the etherpad but it generally looks pretty good. 

I made the point to capitalize features names which we already do in product like but we might want to double check some of that.

Comment 6

[Sebastian Kaspari (:sebastian; :pocmo)]

(In reply to Anthony Lam (:antlam) from comment #5)
> Awesome, thanks Sebastian!
> 
> I left some comments and questions in the etherpad but it generally looks
> pretty good. 
> 
> I made the point to capitalize features names which we already do in product
> like but we might want to double check some of that.

Thank you! I tried to answer all your questions. Feel free to directly edit the titles and descriptions in place. This way it'll be easier to copy them into the project once finalized. :)

Comment 7

[Anthony Lam (:antlam) Find me, @antlam on Slack!]

Talked to Sebastian on IRC, I think we can do without the descriptions.

Comment 8

[Robin Andersen [:tecgirl]]

Agreed, we can do without the descriptions.

Comment 9

[Allison Naaktgeboren :ally]

last call for input! srsly!

Comment 10

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: settings-restrictions.png

Screenshot of the settings with final strings.

Comment 11

[Sebastian Kaspari (:sebastian; :pocmo)]

Attachment: 1189680-strings.patch

Patch with finalized title strings and removed descriptions. Anyone ready for a last-minute review for 42? :)

Comment 12

[:Margaret Leibovic]

Comment on attachment 8644851 [details] [diff] [review]
1189680-strings.patch

Review of attachment 8644851 [details] [diff] [review]:
-----------------------------------------------------------------

Looks good to me, but please rev the entity names before landing.

::: mobile/android/base/locales/en-US/android_strings.dtd
@@ +690,5 @@
>  <!-- Restrictions -->
>  <!-- Localization note: These are restrictions the device owner (e.g. parent) can enable for
>       a restricted profile (e.g. child). Used inside the Android settings UI. -->
> +<!ENTITY restriction_disallow_import_settings_title "Disable \'Import from Android\'">
> +<!ENTITY restriction_disallow_addons_title "Disable add-on installation">

You should rev the entity names if their contents have changed, since some localizers start working on Nightly, and their tools won't detect the new string if the entity name is the same.

Usually we just add a "2" or something to the end, and you can just update it in strings.xml.in, but leave the string resource id alone.

Comment 13

[Sebastian Kaspari (:sebastian; :pocmo)]

Comment on attachment 8644851 [details] [diff] [review]
1189680-strings.patch

Okay. I'll update the entity names.

Comment 14

[Sebastian Kaspari (:sebastian; :pocmo)]

url:        https://hg.mozilla.org/integration/fx-team/rev/0c01ce7ee9feb3a29358e69656fd09cbbac84e7e
changeset:  0c01ce7ee9feb3a29358e69656fd09cbbac84e7e
user:       Sebastian Kaspari <[e-mail]>
date:       Fri Aug 07 18:32:01 2015 +0200
description:
Bug 1189680 - Restricted profile: Finalize title strings and remove descriptions. r=margaret

Comment 15

[Sebastian Kaspari (:sebastian; :pocmo)]

Landed. :)

Comment 16

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/0c01ce7ee9fe

Comment 17

[Catalin Suciu [:csuciu]]

Verified that the new strings are displayed correctly on Nightly and Aurora