Closed
Bug 702319
Opened 13 years ago
Closed 13 years ago
update telemetry opt-in text to cover entire scope of probes
Categories
(Firefox for Android Graveyard :: General, defect, P3)
Firefox for Android Graveyard
General
Tracking
(blocking-fennec1.0 beta+, fennec11+)
RESOLVED
FIXED
Firefox 14
People
(Reporter: geekboy, Assigned: bnicholson)
References
Details
Attachments
(2 files, 2 obsolete files)
(deleted),
patch
|
mfinkle
:
review+
Pike
:
review-
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mfinkle
:
review+
Pike
:
review+
mfinkle
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
In bug 688513 comment 15, mfinkle asked for a bug to revise the telemetry opt-in string.
https://bugzilla.mozilla.org/show_bug.cgi?id=699513#c15
The opt-in string created by the patch in that bug does not clearly cover all the things Telemetry wants to collect, for example hardware attributes or network capabilities.
Here's what landed:
> "Help improve %S by sending anonymous usage information to Mozilla."
We don't want a laundry list, but the shortest string we've come up with that covers our collection in desktop Firefox is documented in bug 685373. My impression is that a wider variety of probes is collected on mobile, and we should provide an opt-in string that gives people an accurate picture of what is being collected without them needing to dig into the privacy policy to be informed.
Picking up here from comment 11 in the previous fennec telemetry opt-in bug:
https://bugzilla.mozilla.org/show_bug.cgi?id=699513#c11
String length here is constrained due to screen size, which is why the desktop Firefox string is not being used here. Mark, what are the size limits?
Updated•13 years ago
|
Assignee: nobody → madhava
Priority: -- → P3
Comment 1•13 years ago
|
||
Unless the information is actually anonymous, we shouldn't call it anonymous. Isn't it arcane and pseudonymous?
Reporter | ||
Comment 2•13 years ago
|
||
I'm still hoping to hear a little bit about constraints on the string here. The string that landed is a bit misleading and not very well informing, so I'd like to know what kind of space we have to work with to craft a better message.
Updated•13 years ago
|
tracking-fennec: --- → 11+
Comment 3•13 years ago
|
||
mfinkle - Can you comment on the string length restrictions? Can we include a link in the string so that we can discuss the possibility of linking to more information (not necessarily directly to the privacy doc). Something like:
"Help improve %S by sending usage information to Mozilla. Learn more."
where Learn more is a link to a mobile specific page (with more space than we have in a door hangar) that contains a short form of the details.
Comment 4•13 years ago
|
||
If it's going to get longer than about a sentence, I think we probably do want to have something short and accurate but not threatening, and and "Learn More" link. The page we take you to should let you opt in there, as well.
Comment 5•13 years ago
|
||
I'd rather have a shorter string which uses general terms to describe a superset of what's being collected, with a "Learn more" link. However, if we can use adjectives to allude to the category of data being collected, that would be ideal. I don't know how frequently people click "learn more", and I'd love to see UX comment on that, or if there are better strings than "Learn more" that see more interaction.
Comment 6•13 years ago
|
||
(In reply to Lawrence Mandel [:lmandel] from comment #3)
> mfinkle - Can you comment on the string length restrictions? Can we include
> a link in the string so that we can discuss the possibility of linking to
> more information (not necessarily directly to the privacy doc). Something
> like:
>
> "Help improve %S by sending usage information to Mozilla. Learn more."
> where Learn more is a link to a mobile specific page (with more space than
> we have in a door hangar) that contains a short form of the details.
We do not have support for links in the native UI doorhangers. We do have support for buttons.
I think one sentence is the maximum we want for string length.
Comment 7•13 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #6)
> We do not have support for links in the native UI doorhangers. We do have
> support for buttons.
Can we have three buttons: Yes, No, Learn More? (If it makes sense I can file a bug to add support for links in mobile door hangars. I think there should be no expectation that this will get into the first native UI release.)
Comment 8•13 years ago
|
||
"Learn more" links in doorhangers are a method that we're going to want to use a significant amount in the future. I'm not happy with a button because I don't think that they have the right user-experience for this feature. I think that we should add support for links in doorhangers, and use a button until that's ready.
Comment 9•13 years ago
|
||
(In reply to Lawrence Mandel [:lmandel] from comment #7)
> (In reply to Mark Finkle (:mfinkle) from comment #6)
> > We do not have support for links in the native UI doorhangers. We do have
> > support for buttons.
>
> Can we have three buttons: Yes, No, Learn More? (If it makes sense I can
> file a bug to add support for links in mobile door hangars. I think there
> should be no expectation that this will get into the first native UI
> release.)
Three buttons means it will be very tight in portrait mode. Instead of "Learn More", you might consider alternatives. Maybe "Help" or "Info" - something that won't expand too much for l10n.
Comment 10•13 years ago
|
||
Then let's aim for a "Learn more" link once that feature is ready, and go with Yes/No/Info until then.
Comment 11•13 years ago
|
||
Also, string proposal:
"Send info to Mozilla to that we can fix and improve %s."
Comment 12•13 years ago
|
||
(In reply to Tom Lowenthal [:StrangeCharm] from comment #11)
> Also, string proposal:
>
> "Send info to Mozilla to that we can fix and improve %s."
Fixing is improving right. :) How about just
"Send info to Mozilla so that we can improve %s."
This would go along with yes, no, info buttons, where the info button will be replaced with a Learn more link once links are supported in mobile door hangars.
Updated•13 years ago
|
blocking-fennec1.0: --- → ?
Updated•13 years ago
|
Assignee: madhava → bnicholson
blocking-fennec1.0: ? → beta+
Assignee | ||
Comment 13•13 years ago
|
||
Do we have a page to link to from the "Learn more" link?
Comment 14•13 years ago
|
||
Assignee | ||
Comment 15•13 years ago
|
||
In desktop, we append "how-can-i-help-submitting-performance-data" to "app.support.baseURL" (http://mxr.mozilla.org/mozilla-central/source/browser/components/nsBrowserGlue.js#834). We have a different baseURL on mobile, so doing the same thing yielded an invalid URL. I created the "browser.telemetry.infoURL" preference which contains the telemetry "Learn more" URL.
We'll need to change this page (or point to a different one) since it uses desktop Firefox in the opt-in/out screenshots.
Attachment #603523 -
Flags: review?(mark.finkle)
Assignee | ||
Comment 16•13 years ago
|
||
I've updated the patch to use the server owner rather than hard-coding "Mozilla".
I'm not sure what to call this pref. We already have a "toolkit.telemetry.infoURL", but thats points to http://www.mozilla.com/legal/privacy/firefox.html#telemetry.
Attachment #603523 -
Attachment is obsolete: true
Attachment #603523 -
Flags: review?(mark.finkle)
Attachment #603557 -
Flags: review?(mark.finkle)
Comment 17•13 years ago
|
||
Sid, Tom - The current patch uses the string that I proposed in comment 12. This is not as descriptive as the string we use on desktop but is much shorter for the mobile space restrictions. Concerns?
"Send info to %S so that we can improve %S?"
Comment 18•13 years ago
|
||
(In reply to Brian Nicholson (:bnicholson) from comment #15)
> Created attachment 603523 [details] [diff] [review]
> patch
>
> In desktop, we append "how-can-i-help-submitting-performance-data" to
> "app.support.baseURL"
> (http://mxr.mozilla.org/mozilla-central/source/browser/components/
> nsBrowserGlue.js#834). We have a different baseURL on mobile, so doing the
> same thing yielded an invalid URL. I created the "browser.telemetry.infoURL"
> preference which contains the telemetry "Learn more" URL.
>
> We'll need to change this page (or point to a different one) since it uses
> desktop Firefox in the opt-in/out screenshots.
I have created a Telemetry article that is specific to mobile. Once approved the URL will be
https://support.mozilla.org/kb/how-can-i-help-submitting-mobile-performance-data
Comment 19•13 years ago
|
||
Bug 725546 has promise for "fixing" the app.support.baseURL issue, but I can't get it to work yet.
Let's try to use that approach though, instead of making "browser.telemetry.infoURL" just to be consistent with desktop.
Comment 20•13 years ago
|
||
Comment on attachment 603557 [details] [diff] [review]
patch v2
This is fine except for the support URL. We really should wait to land this until we get that sorted.
Attachment #603557 -
Flags: review?(mark.finkle) → review+
Updated•13 years ago
|
blocking-fennec1.0: beta+ → +
Updated•13 years ago
|
blocking-fennec1.0: + → beta+
Comment 21•13 years ago
|
||
The SUMO article for mobile has been published at
https://support.mozilla.org/kb/how-can-i-help-submitting-mobile-performance-data
Comment 22•13 years ago
|
||
(In reply to Lawrence Mandel [:lmandel] from comment #21)
> The SUMO article for mobile has been published at
> https://support.mozilla.org/kb/how-can-i-help-submitting-mobile-performance-
> data
Can we get this URL turned into a Firefox UI friendly link? As mentioned in bug 725546. That will allow us to switch to the better app.support.baseURL
Assignee | ||
Comment 23•13 years ago
|
||
Updated the URL to point to baseURL + "how-can-i-help-submitting-mobile-performance-data", which currently 404s. The assumption is that this URL will eventually redirect to the correct URL (https://support.mozilla.org/kb/how-can-i-help-submitting-mobile-performance-data).
Attachment #603557 -
Attachment is obsolete: true
Attachment #604817 -
Flags: review?(mark.finkle)
Comment 24•13 years ago
|
||
(In reply to Mark Finkle (:mfinkle) from comment #22)
> (In reply to Lawrence Mandel [:lmandel] from comment #21)
> > The SUMO article for mobile has been published at
> > https://support.mozilla.org/kb/how-can-i-help-submitting-mobile-performance-
> > data
>
> Can we get this URL turned into a Firefox UI friendly link? As mentioned in
> bug 725546. That will allow us to switch to the better app.support.baseURL
cc Michelle Luna from SUMO mobile for help with the Firefox UI friendly link.
Comment 25•13 years ago
|
||
Comment on attachment 604817 [details] [diff] [review]
patch v3
Let's land this and updated later if needed
Attachment #604817 -
Flags: review?(mark.finkle) → review+
Comment 26•13 years ago
|
||
The topics names can (should?) be the same across different platforms and we can target different articles based on product among other things (locale, platform, version, etc). I just added a redirect for the "how-can-i-help-submitting-performance-data" topic that goes to the mobile version of the article when the product is mobile.
For example:
https://support.mozilla.org/1/mobile/13.0a1/Darwin/en-US/how-can-i-help-submitting-performance-data
Goes to:
https://support.mozilla.org/en-US/kb/how-can-i-help-submitting-mobile-performance-data?as=u
Comment 27•13 years ago
|
||
Brian - make sure to update your patch before checkin to point to the URL in comment 26
Assignee | ||
Comment 28•13 years ago
|
||
http://hg.mozilla.org/integration/mozilla-inbound/rev/29be02f524cf
I updated the baseURL to point to http://support.mozilla.org/1/mobile/%VERSION%/%OS%/%LOCALE%/. This is now parallel to the desktop baseURL (http://support.mozilla.org/1/firefox/%VERSION%/%OS%/%LOCALE%/), and it works for the URL given in comment 26 and the Support link we currently have on the about: page.
Comment 29•13 years ago
|
||
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 13
Comment 30•13 years ago
|
||
Why/how exactly did you approve a patch that changes significantly a string (telemetry.optin.message) adding a new variable without changing the entity name? And to create even more confusion this change landed just before the move to aurora, so now the problem is on two different repositories.
Comment 31•13 years ago
|
||
Comment on attachment 604817 [details] [diff] [review]
patch v3
Review of attachment 604817 [details] [diff] [review]:
-----------------------------------------------------------------
::: mobile/android/locales/en-US/chrome/browser.properties
@@ +56,5 @@
> blockPopups.label=Block Popups
>
> # Telemetry
> +telemetry.optin.message=Send info to %S so that we can improve %S?
> +telemetry.optin.learnMore=Learn more
Yeah, this should have changed the key, and also, added a comment what those place holders actually stand for. Also, it' good to just use %1$S and %2$S right away, if you're using more than one parameter. Helps also to document what's what.
Attachment #604817 -
Flags: review-
Assignee | ||
Updated•13 years ago
|
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Assignee | ||
Comment 32•13 years ago
|
||
[Approval Request Comment]
Fixes l10n issues described in comment 30 and comment 31.
Attachment #605850 -
Flags: review?(mark.finkle)
Attachment #605850 -
Flags: review?(community)
Attachment #605850 -
Flags: approval-mozilla-aurora?
Assignee | ||
Updated•13 years ago
|
Attachment #605850 -
Flags: review?(community) → review?(l10n)
Updated•13 years ago
|
Attachment #605850 -
Flags: review?(mark.finkle)
Attachment #605850 -
Flags: review+
Attachment #605850 -
Flags: approval-mozilla-aurora?
Attachment #605850 -
Flags: approval-mozilla-aurora+
Comment 33•13 years ago
|
||
Comment on attachment 605850 [details] [diff] [review]
follow-up patch
Review of attachment 605850 [details] [diff] [review]:
-----------------------------------------------------------------
r=me, too, thanks.
Attachment #605850 -
Flags: review?(l10n) → review+
Assignee | ||
Comment 34•13 years ago
|
||
Assignee | ||
Comment 35•13 years ago
|
||
Comment 36•13 years ago
|
||
Status: REOPENED → RESOLVED
Closed: 13 years ago → 13 years ago
Resolution: --- → FIXED
Target Milestone: Firefox 13 → Firefox 14
Updated•12 years ago
|
Keywords: privacy-review-needed
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•