1114476 - [DT][Privacy]change Privacy Panel to a meaningful word

Status: RESOLVED FIXED

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

Description

[[e-mail]]

Privacy Panel is a too general word, 
I can't understand what Privacy Panel/Browsing Privacy/Do not track denotes.

Maybe we could change 'Privacy Panel' to 'Location & Remote' or something really help users to check them.

Comment 1

[[e-mail]]

@jenny please suggest a good name once you have the bandwidth, thanks!

Comment 2

[Jenny Lee]

Hi Fred,
I agree the naming is a little confusing. Let's involve UX designer from DT as Privacy panel is their topic.
Thanks for pointing that out!

Hi Wesley,
Could you kindly help adding UX designer from DT for this bug? Thanks!

Comment 3

[Wesley Huang [:wesley_huang] (EPM) (NI me)]

[NI to Wilfred as he's the contact window to partner]
[NI to Josh who will be the EPM for privacy feature]

Comment 4

[Wilfred Mathanaraj [:WDM]]

I think we have had extensive discussions about the naming of this feature contribution. I have added Alina from our Privacy team and MArta from DT side to comment further. But I dont think this is something we should focus on for 2.2.

Comment 5

[marta]

The privacy panel was meant as a combination of all privacy settings, something that will grow with time to combine more features. I do agree that it would make sense to merge all of it together.

Comment 6

[[e-mail]]

@marta, do you have suggestion for v2.2 (or future version) to make current settings list looks more meaningful to end user?

Comment 7

[marta]

Privacy Controls? Putting in the loop other DT people.

Comment 8

[[e-mail]]

'Privacy controls' +1!

According to https://wiki.mozilla.org/Release_Management/FirefoxOS/2_2_Schedule , for 2.2 most strings should have landed by 2015/02/23, so we'd better come out the decision on time.

Comment 9

[Josh Cheng [:josh]]

Is anyone from like to comment about the naming here? 
Do we really want to rename Privacy Panel on 2.2?

Comment 10

[[e-mail]]

@jenny if you think 'Privacy controls' is good to go, I could provide PR for this issue.

Comment 11

[Théo Chevalier]

I'm afraid "Privacy controls" will be truncated on the homescreen App icon. Has anyone checked?

Comment 12

[Jenny Lee]

Yea sure, let's go!

Comment 13

[[e-mail]]

@theo I think privacy panel does not shown on the homescreen App anymore (at least on master), but only accessible via settings.

Comment 14

[GitHub Autolander Bot]

Attachment: [PullReq] gasolin:issue-1114476 to mozilla-b2g:master

Comment 15

[[e-mail]]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

Naming changed from `Privacy Panel` to `Privacy Controls`. The change only affect on wording inside the app and the settings, but not the module name.

also set ui review to @jenny

Comment 16

[Jenny Lee]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

looks good =)

Comment 17

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

Please also change the string id when changing a string, thanks.

Comment 18

[[e-mail]]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

Thanks for notice! changed string id and make sure they looks fine on device.

Please kindly review it again.

Comment 19

[Arthur Chen [:arthurcc][inactive after 6/30]]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

The code looks good to me. Let's wait for the inputs from DT people.

Comment 20

[[e-mail]]

merged https://github.com/mozilla-b2g/gaia/commit/b077b97eead06bac34a7ba95eb2d77650a97ee2b

thanks!

Comment 21

[[e-mail]]

Comment on attachment 8561251 [details] [review]
[PullReq] gasolin:issue-1114476 to mozilla-b2g:master

[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): bug 1057675 ([meta]privacy_panel)
[User impact] if declined: User will see Privacy panel instead of Privacy controls, which is not meaningful to users
[Testing completed]: Treeherder green
[Risk to taking this patch] (and alternatives if risky): None
[String changes made]: yes

Comment 22

[GitHub Autolander Bot]

Attachment: [PullReq] gasolin:issue-1114476-22 to mozilla-b2g:v2.2

Comment 23

[[e-mail]]

uplifted to 2.2 https://github.com/mozilla-b2g/gaia/commit/bc28ee339084f09e6be74302aecf76a2f9927930

Comment 24

[Stefan Plewako [:stef]]

This didn't completely removed all occurrences of privacy panel…

Comment 25

[[e-mail]]

stefan could you help point out where the 'occurrences' are, so we could fix it.

Comment 26

[Stefan Plewako [:stef]]

http://transvision.mozfr.org/?recherche=privacy+panel&repo=gaia&sourcelocale=en-US&locale=en-US&search_type=strings

Comment 27

[[e-mail]]

@stefan it's intended because the error message or the description are not usually seem by user. I think it worth to keep them as a clue for developer since the module name in gaia folder is still 'apps/privacy-panel'.

Comment 28

[Stefan Plewako [:stef]]

The reasoning here seems so wrong that I'm reopening this bug.

Comment 29

[[e-mail]]

As comment 27, I think it's not make sense to reopen this bug.
If you feel it's the bug, please open a follow up issue and nominate it.

Comment 30

[Stefan Plewako [:stef]]

(In reply to Fred Lin [:gasolin] from comment #27)
> @stefan it's intended because the error message or the description are not
> usually seem by user. I think it worth to keep them as a clue for developer
> since the module name in gaia folder is still 'apps/privacy-panel'.

Error messages are for users, debugging is for developers, source code comments are clues for developers and this bug was solely about renaming Privacy Panel to something meaningful for users.

Comment 31

[[e-mail]]

@Stefan REOPEN bug means we would backout the commit, I think it's not the purpose you want to achieve.

I've fired bug 1141971 that aim to fix the remain wording. Hope it serves your purpose well.

Comment 32

[Stefan Plewako [:stef]]

(In reply to Fred Lin [:gasolin] from comment #31)
> @Stefan REOPEN bug means we would backout the commit, I think it's not the
> purpose you want to achieve.

No, it means „This bug was once resolved, but the resolution was deemed incorrect”, https://bugzilla.mozilla.org/page.cgi?id=fields.html#status and for sure it doesn't define how to accomplish agreement and correct resolution.

> I've fired bug 1141971 that aim to fix the remain wording. Hope it serves
> your purpose well.

OK, fine.