Clarify "Automatic updates" under Home in Settings

RESOLVED FIXED in Firefox 48

Status

()

defect
RESOLVED FIXED
4 years ago
3 years ago

People

(Reporter: kbrosnan, Assigned: Grisha)

Tracking

Trunk
Firefox 48
Unspecified
Android
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox48 verified, fennec48+)

Details

Attachments

(1 attachment)

Settings > General > Home > scroll to the bottom > "Automated updates"
Settings > Advanced > "Automatic updates"

Do we need two copies of this pref?
Hm, according to https://bugzilla.mozilla.org/show_bug.cgi?id=1216257#c4, it should be under Advanced as "Automatic updates".

Not sure what the one in General is, I'm not seeing it on my 6P
(In reply to Kevin Brosnan [:kbrosnan] from comment #0)
> Settings > General > Home > scroll to the bottom > "Automated updates"

This setting is not about app updates, it's about content updates for content in home panel add-ons. Given that this is confusing, and that this feature is only half-baked, I think we should just remove this.

I don't believe this is a regression in 45, since I see this item in Customize -> Home.
Flags: needinfo?(alam)
45 is where the settings redesign landed. Lets just clear the affected flags then.
(In reply to :Margaret Leibovic from comment #2)
> (In reply to Kevin Brosnan [:kbrosnan] from comment #0)
> > Settings > General > Home > scroll to the bottom > "Automated updates"
> 
> This setting is not about app updates, it's about content updates for
> content in home panel add-ons. Given that this is confusing, and that this
> feature is only half-baked, I think we should just remove this.

Right, I got confused - removing this WFM.
Flags: needinfo?(alam)
Assignee: nobody → margaret.leibovic
tracking-fennec: ? → 45+
Summary: Settings lists updates in multiple locations → Remove home panel content update settings item
I see that a decent number of people hit this settings item ("home.sync.updateMode") in UI telemetry... but I'm not sure if they know what it means or does. It will only do something useful if you have a home panel add-on installed, and if that add-on is using the HomeProvider `requestSync` and `addPeriodicSync` APIs.

If we're going to remove this pref from the UI, we should probably remove this functionality as well.

Barbara, how do you feel about this from a product perspective? Should we remove this part of our home panels add-on API? Or maybe instead what we need to do is make this settings item clearer, since obviously right now it's easy to confuse it with app updates.

I could add telemetry to see if add-on developers are using these APIs, but I'm going to guess they're used a decent amount, since my home panel add-on boilerplate includes them. However, home panel add-ons themselves are not that popular, so maybe this isn't something we care to support. But then that opens up a whole other can of worms about the future of home panel add-on support.

It just seems like it would be more work to remove it than to fix the confusion problem.
Flags: needinfo?(bbermes)
> It just seems like it would be more work to remove it than to fix the
> confusion problem.

It seems to me easier to update the string than to start removing this and all its dependencies.

Why don't we update the string to a more concrete message and add a probe to monitor. We can always remove it later.

Suggestions for ...->home->: "Automated content updates"

As for the app updates -- what kind if updates are we talking about?
Flags: needinfo?(bbermes)
(In reply to Barbara Bermes [:barbara] from comment #6)
> > It just seems like it would be more work to remove it than to fix the
> > confusion problem.
> 
> It seems to me easier to update the string than to start removing this and
> all its dependencies.
> 
> Why don't we update the string to a more concrete message and add a probe to
> monitor. We can always remove it later.
> 
> Suggestions for ...->home->: "Automated content updates"

Okay, I think this makes sense, too. I'm going to make this bug track 47 instead of 45, since this will require a string change. Anthony, how do you feel about this?

> As for the app updates -- what kind if updates are we talking about?

For the app updates, we're talking about the updates Mozilla serves for Firefox itself (e.g. for Nightly, Aurora updates). Unfortunately, the fact that we always build the Firefox updater with our code, regardless of whether updates come from Mozilla or the Play Store, means that we always show this pref, even if we never download app updates from Mozilla.

I thought there was a bug filed about this, but I can't find it. I don't think it's actively harmful, since we'll never be downloading those updates, so we're respecting the user's wish to only download over wifi or never download.
tracking-fennec: 45+ → 47+
Flags: needinfo?(alam)
(In reply to :Margaret Leibovic from comment #7)
> (In reply to Barbara Bermes [:barbara] from comment #6)
> > > It just seems like it would be more work to remove it than to fix the
> > > confusion problem.
> > 
> > It seems to me easier to update the string than to start removing this and
> > all its dependencies.
> > 
> > Why don't we update the string to a more concrete message and add a probe to
> > monitor. We can always remove it later.
> > 
> > Suggestions for ...->home->: "Automated content updates"
> 
> Okay, I think this makes sense, too. I'm going to make this bug track 47
> instead of 45, since this will require a string change. Anthony, how do you
> feel about this?

WFM!

Let's change this language right now. 

I would also like to know if this is a nuance we could remove or we should make a bigger deal out of for the Add-ons work later down the line. So, we should monitor the telemetry on this after the copy changes too.

Why don't we call it "Content updates" instead? that makes more sense I think

+-------------------------------------------------+
|                                                 |
|  Content updates                                |
|  Enabled                                        |
|                                                 |
+-------------------------------------------------+
Flags: needinfo?(alam) → needinfo?(margaret.leibovic)
Can we clarify that this setting is only for content from add-ons? I don't want users to be confused by the fact that everything in your default home panels is local, so what content is being updated? Or maybe they would think this relates to sync, but that isn't the case.
Flags: needinfo?(margaret.leibovic)
Flags: needinfo?(bbermes)
Flags: needinfo?(alam)
If the current string is ambiguous, and the update only applies to add-ons, we could call it

Content updates from Add-Ons

or

Updates of Add-ons Content
...
Flags: needinfo?(bbermes)
Hm... I don't think any of these copy changes will really make it obvious enough that this setting is about Add-ons only. The best non-scope creeping, immediately do-able solution is to rename this for now. 

But, I've filed bug 1249729 to keep track of a more permanent solution.

(In reply to :Margaret Leibovic from comment #9)
> Can we clarify that this setting is only for content from add-ons? I don't
> want users to be confused by the fact that everything in your default home
> panels is local, so what content is being updated? Or maybe they would think
> this relates to sync, but that isn't the case.

It's currently under the header "Content Settings", why don't we just create another header called "Add-ons" and put this pref there?

Content settings

+-------------------------------------------------+
|                                                 |
|  Show site suggestions                          |
|  Display shortcuts to sites on your homep..     |
|                                                 |
+-------------------------------------------------+

Add-ons

+-------------------------------------------------+
|                                                 |
|  Content updates                                |
|  Enabled                                        |
|                                                 |
+-------------------------------------------------+

BTW, what does "show site suggestions" do again?
Flags: needinfo?(alam) → needinfo?(margaret.leibovic)
Summary: Remove home panel content update settings item → Clarify "Automatic updates" under Home in Settings
(In reply to Anthony Lam (:antlam) from comment #11)

> BTW, what does "show site suggestions" do again?

It shows suggested sites, i.e. the tiles we show on first run.
Flags: needinfo?(margaret.leibovic)
Oops, I missed the merge, this will have to happen in 48.
tracking-fennec: 47+ → 48+
Assignee: margaret.leibovic → gkruglov
Attachment #8728113 - Attachment description: MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings → MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings r=margaret
Attachment #8728113 - Flags: review?(margaret.leibovic)
Comment on attachment 8728113 [details]
MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings r=margaret

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38793/diff/1-2/
I see the patch here updates this string to say "Content updates", but I feel like it would be clearer if this said something like "Content updates from add-ons". antlam, what do you think?
Flags: needinfo?(alam)
Lets use the subtitle that we talked about in comment 11 to make this distinction.

Saying "Add-ons" up there will allow us to stick with "Content updates" as the pref title.
Flags: needinfo?(alam)
Will make that change and re-push.
Comment on attachment 8728113 [details]
MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings r=margaret

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38793/diff/2-3/
Blocks: 1249783
Comment on attachment 8728113 [details]
MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings r=margaret

https://reviewboard.mozilla.org/r/38793/#review35565

Looks good! You should re-push with my one comment addressed, add the "checkin-needed" keyword to the bug, and then a sheriff will land this for you.

::: mobile/android/base/resources/xml/preferences_home.xml:40
(Diff revision 3)
> -
> +                        

Nit: Remove trailing whitespace (you should be able to set a preference in your editor to avoid saving trailing whitespace).
Attachment #8728113 - Flags: review?(margaret.leibovic) → review+
Comment on attachment 8728113 [details]
MozReview Request: Bug 1248046 - clarify "content updates" label under Home in Settings r=margaret

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/38793/diff/3-4/
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/fd53c862d17a
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 48
"Automatic updates - Only over Wi-Fi" is displayed in Settings > Advanced 
"Content updates - Enabled" is displayed in Settings > General > Home > Add-ons

Verified as fixed using:
Device: One A2001 (Android 5.1.1) 
Build: Firefox for Android 48.0a1 (2016-03-13)
You need to log in before you can comment on or make changes to this bug.