802899 - [settings] "Applications Permissions" top-level item should be renamed per the spec

Status: VERIFIED FIXED

(Firefox OS Graveyard :: Gaia::Settings, defect, P1)

Description

[Dietrich Ayala (:dietrich)]

per the visual designs, should be "App Permissions":

https://www.dropbox.com/sh/pygek1oa9a5pspu/3DPdPEEK0y/Visual%20Design/Apps/Settings

but the ux specs say "Apps":

https://www.dropbox.com/sh/pygek1oa9a5pspu/L4nVdWDYAs/Interaction%20Design/Apps/Settings/R1_Security_and_Privacy_v3.pdf

Josh, Patryk: Which is correct?

Comment 1

[Dietrich Ayala (:dietrich)]

app install specs also say "Apps": https://www.dropbox.com/sh/b0kyykhzcfkpm8b/fdA0gFkf5U/Gaia_AppInstall_20121011.pdf

Comment 3

[Daniel Holbert [:dholbert]]

Note that we don't list any actual permissions in this menu, as far as I've seen, so (as noted in comment 1) "Apps" is a much more suitable title.

Shouldn't this be blocking? ( marking bb? )

Also, I think this would be a late-l10n change -- adding that keyword & CC'ing stas.

Comment 4

[Fabien Cazenave [:kaze]]

Attachment: patch proposal

Using “App Permissions” in the root panel and in the related sub-panel to match the spec. Let me know if we should use “Applications” instead (I’d find that confusing but I’m not a UX engineer).

Comment 5

[Staś Małolepszy :stas]

Comment on attachment 684640 [details]
patch proposal

Josh -- see comment 0 and comment 1.  Which one do you prefer?

Comment 6

[Josh Carpenter [:jcarpenter]]

[cc'ing Larissa for input, but I think she's away]

"App permissions" will be fine for v1. It's a bit too narrow for the functionality that the section actually enables (per app: clear data, edit permissions and uninstall) but it describes the primary use case (permissions review/edit), and is contextually appropriate within the parent section (Security & Privacy). 

In future versions we'll probably break apps out into a top-level section. For v1, this will work. 

Thanks!

Comment 7

[Staś Małolepszy :stas]

Thanks everyone!

https://github.com/mozilla-b2g/gaia/commit/859fb6be47b5e644c6f24955071afe15b07b8192

Comment 8

[Larissa Co [:lco]]

Agree with(In reply to Josh Carpenter [:jcarpenter] from comment #6)
> [cc'ing Larissa for input, but I think she's away]
> 
> "App permissions" will be fine for v1. It's a bit too narrow for the
> functionality that the section actually enables (per app: clear data, edit
> permissions and uninstall) but it describes the primary use case
> (permissions review/edit), and is contextually appropriate within the parent
> section (Security & Privacy). 
> 
> In future versions we'll probably break apps out into a top-level section.
> For v1, this will work. 
> 
> Thanks!

I agree with Josh's comments. Thanks!