1199354 - Update the Android and iOS download links on the Sync preferences pane

Status: VERIFIED FIXED

(Firefox :: Settings UI, defect, P1)

Description

[Chris Karlof [:ckarlof]]

For context, see https://bugzilla.mozilla.org/show_bug.cgi?id=1149721#c49

Since bug will also replace bug 1194508, which was for adding the iOS solicitation to preferences. 

Here are the visual designs:

signed out: https://bug1149721.bmoattachments.org/attachment.cgi?id=8636338 
signed in: https://bug1149721.bmoattachments.org/attachment.cgi?id=8636339

Here are the link targets:

Android:

https://www.mozilla.org/firefox/android/?utm_source=firefox-browser&utm_medium=firefox-browser&utm_campaign=sync-preferences

iOS:

https://www.mozilla.org/firefox/ios/?utm_source=firefox-browser&utm_medium=firefox-browser&utm_campaign=sync-preferences

Comment 2

[Chris Karlof [:ckarlof]]

Bug 1194508 was originally for adding the iOS solicitation to the Sync prefs pane. I've marked that as a duplicate of this.

Comment 3

[Mark Hammond [:markh] [:mhammond]]

Attachment: 0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

This makes the requested changes. I'm unsure of the "attribution" for the iOS trademark, so I'll ping legal for help (it's complicated - Cisco owns that trademark!).

Flod, in the meantime, would you mind checking the sanity of this from a l10n perspective?

Comment 4

[Mark Hammond [:markh] [:mhammond]]

Attachment: ios-link-signedout.png

How it looks in the "not signed in" state.

Comment 5

[Mark Hammond [:markh] [:mhammond]]

Comment on attachment 8662762 [details]
ios-link-signedout.png

Bryan, are you happy with this?

Comment 6

[Mark Hammond [:markh] [:mhammond]]

Attachment: ios-link-signedin.png

How it looks when signed in.

Comment 7

[Mark Hammond [:markh] [:mhammond]]

I opened bug 1205955 to request legal review of the use of the iOS trademark and attribution - I hope that's the right place :)

Comment 8

[Francesco Lodolo [:flod]]

Comment on attachment 8662761 [details] [diff] [review]
0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

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

Nits on comments, but the structure looks good.

::: browser/locales/en-US/chrome/browser/preferences/sync.dtd
@@ +89,5 @@
>  <!ENTITY signedOut.accountBox.create  "Create Account">
>  <!ENTITY signedOut.accountBox.signin  "Sign In">
>  
>  <!ENTITY signedIn.engines.label       "Sync across all devices">
>  

Given how complex this piece is becoming (I wish we could use .properties and one string), I'd put a comment explaining how the final string will look.

<!-- LOCALIZATION NOTE (mobilePromo.*): the following strings will be used to 
     create a single sentence with active links.
     The resulting sentence in English is: "Download Firefox for Android or 
     iOS to sync with your mobile device." -->

@@ +91,5 @@
>  
>  <!ENTITY signedIn.engines.label       "Sync across all devices">
>  
>  <!ENTITY mobilePromo.start            "Download Firefox for ">
> +<!-- LOCALIZATION NOTE (mobilePromo.androidLink): This is a link title that links to https://www.mozilla.org/firefox/android/?... -->

Editor fault (end of the line cut)?

I'd say "Used as text for a link to https://www.mozilla.org/firefox/android/"

@@ +96,3 @@
>  <!ENTITY mobilePromo.androidLink      "Android™">
> +
> +<!-- LOCALIZATION NOTE (mobilePromo.iOSBefore): This is text displayed between the mobilePromo.androidLink and the iOS icon -->

iOS icon -> mobilePromo.iOSLink

"This text is displayed between mobilePromo.androidLink and mobilePromo.iOSLink"

@@ +96,5 @@
>  <!ENTITY mobilePromo.androidLink      "Android™">
> +
> +<!-- LOCALIZATION NOTE (mobilePromo.iOSBefore): This is text displayed between the mobilePromo.androidLink and the iOS icon -->
> +<!ENTITY mobilePromo.iOSBefore         " or ">
> +<!-- LOCALIZATION NOTE (mobilePromo.iosLink): This is a link title that links to https://www.mozilla.org/firefox/ios/?... -->

Same as above: "Used as text for a link to https://www.mozilla.org/firefox/ios/"

Comment 9

[Mark Hammond [:markh] [:mhammond]]

Attachment: 0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

Awesome, thanks very much, all feedback addressed - now to wait on legal...

Comment 10

[Mark Hammond [:markh] [:mhammond]]

The advice from legal is:

> We can't use the Apple logo so you'll have to remove that. You should also
> remove the TM from iOS and change the trademark attribution statement to: "IOS
> is a trademark or registered trademark of Cisco in the U.S. and other countries
> and is used by Apple under license."

Comment 11

[Mark Hammond [:markh] [:mhammond]]

Attachment: 0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

New patch following advice from legal.

Comment 12

[Mark Hammond [:markh] [:mhammond]]

Attachment: ios-link-signedin.png

This is how it looks without the Apple logo. I'm a little concerned that without the logo users may not associate "iOS" with their apple device, so I'd think different wording might be appropriate. If you do come up with better wording that uses an Apple trademark (such as "Apple";), you should probably ask for further advice in bug 1205955 :(

Comment 13

[Mark Hammond [:markh] [:mhammond]]

Attachment: ios-link-signedout.png

For completeness, how it looks when signed out.

Comment 14

[Mark Hammond [:markh] [:mhammond]]

Comment on attachment 8667631 [details]
ios-link-signedin.png

Oops, I think I meant to ask for ui-review from Bryan but I'll leave the request on Ryan too - first one to respond wins :)

Comment 15

[Ryan Feeley [:rfeeley]]

I have confirmation from Urmika in July that we can indeed use the Apple logo. A quick glance at both Dropbox and Spotify shows them using the logos this way in a very similar context without attribution. What has changed?

Comment 16

[Mark Hammond [:markh] [:mhammond]]

I've added that information to the legal bug and indicated we intend to move ahead using the logo. I'll give it a day or 2 then reinstate the earlier patch.

Comment 17

[Chris Karlof [:ckarlof]]

Assigning to Bill to help resolve this with Legal.

Comment 18

[Bill Maggs (bmaggs)]

The meeting with Liz Compton included her review of sites (she named Youtube and several others) that showed that many companies are following the Apple iOS branding guidelines and so we have two choices: use the buttons for Android and iOS app stores; or follow the guidelines and use the product names, both choices linking to the download page in each store. It is worth pointing out that only Android allows download to device from a web page. 

It is true that these choices would lead to not using the robot logo, which Google allows. I'd like to know from Ryan F if the loss of this visually appealing item is a big one, and also which of the two options above (store logo button or product name in a link) is preferable.

Comment 19

[Chris Karlof [:ckarlof]]

> I'd like to know from Ryan F if the loss of this visually appealing item is a big one, and also which of the two options above (store logo button or product name in a link) is preferable.

Paging Ryan F.

Comment 20

[Ryan Feeley [:rfeeley]]

It is a big one and we have user testing to prove that users notice and act on the logos. We have legal clearance from Mika to use the logos as other properties do. Please forge ahead.

Comment 21

[ewong]

continuing to work with Legal - discussion escalated.

Comment 22

[Edwin Wong [:edwong]]

Decision for till legal decides on logo: 

Sync to your mobile device. Download __Firefox for Android__ or __Firefox for iOS__.

*note: no android logo and '__' means it's a link.

Comment 23

[Mark Hammond [:markh] [:mhammond]]

Attachment: ios-link-no-logos.png

Here's a version with no images and no attribution. I felt the separators looked a little strange with all that removed, so I moved these links into the "device" section - IMO it makes sense (it's saying both "name this device" and "get it for your other devices") and it looks a little better.

Ryan, you OK with that?

Comment 24

[Mark Hammond [:markh] [:mhammond]]

Attachment: 0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

Hi Drew,
  We've had a few issues with legal giving the OK for using the Apple logo, and while we hope to eventually get it, we decided to remove the Android logo in the meantime to make things consistent.

We also changed the links to refer to our product names (Firefox for Android and Firefox for iOS), so we removed the attribution for Android and haven't added one for iOS. This is consistent with the landing page for those links, where those terms are also used without attribution.

We'd love to get this in 44.

(Re the screenshot - I've tweaked it so the space before the trailing period is no longer there.)

Comment 25

[Mark Hammond [:markh] [:mhammond]]

Comment on attachment 8680963 [details]
ios-link-no-logos.png

Ryan approved this over IRC.

Comment 26

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/d924e89139c2

Comment 27

[Francesco Lodolo [:flod]]

Comment on attachment 8680977 [details] [diff] [review]
0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

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

Please back-out this, or fix it before it merges to mozilla-central

You basically need to change all the string IDs, you can't update an existing string like this.
https://developer.mozilla.org/en-US/docs/Mozilla/Localization/Localization_content_best_practices#Changing_existing_strings

Comment 28

[Mark Hammond [:markh] [:mhammond]]

Sorry about that, I should have known better. Backed out in https://hg.mozilla.org/integration/fx-team/rev/a7b87003fc4f

Comment 29

[Mark Hammond [:markh] [:mhammond]]

Attachment: 0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

As relanded

Approval Request Comment
[Feature/regressing bug #]: N/A
[User impact if declined]: They wont be informed of the great opportunity to download Firefox for their mobile device on the Sync prefs page! :)
[Describe test coverage new/current, TreeHerder]: Existing tests pass
[Risks and why]: Limited to Sync prefs page, and just adds a new link
[String/UUID change made/needed]: New promo strings, missed the unexpected merge.

Comment 30

[Pulsebot]

https://hg.mozilla.org/integration/fx-team/rev/fc898522b3fb

Comment 31

[Francesco Lodolo [:flod]]

Note from l10n: this seems like a good bug to take. The last thing I'd want is legal pushing to fix things later in the trains.

Comment 32

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/mozilla-central/rev/d924e89139c2

Comment 33

[Mark Hammond [:markh] [:mhammond]]

(In reply to Carsten Book [:Tomcat] from comment #32)
> https://hg.mozilla.org/mozilla-central/rev/d924e89139c2

That was the backout; reopening until the relanded version merges.

Comment 34

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

https://hg.mozilla.org/mozilla-central/rev/fc898522b3fb

Comment 35

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Mark, is this ready to land on Aurora? I noticed comment 33 talks about a back out and therefore wanted to confirm with you before approving this patch.

Comment 36

[Chris Karlof [:ckarlof]]

Ryan, the guidance we received from Nick N. is that from a product perspective he's fine with the approach of using the logos as long as we can instrument this benefit, so that we can make more informed decisions here in the future.

Ryan, is there a previous A/B test which demonstrates this benefit? If not, we'll need to design a new one for this situation if we want to drive this forward.

Comment 37

[Chris Karlof [:ckarlof]]

I filed Bug 1221294 as a followup here to add the logos. If we add them, we'll need to ship them with an A/B test to evaluate their benefit.

Comment 38

[Mark Hammond [:markh] [:mhammond]]

(In reply to Ritu Kothari (:ritu) from comment #35)
> Mark, is this ready to land on Aurora? I noticed comment 33 talks about a
> back out and therefore wanted to confirm with you before approving this
> patch.

Yes, please uplift the relanded patch, https://hg.mozilla.org/mozilla-central/rev/fc898522b3fb

Comment 39

[Francesco Lodolo [:flod]]

Given that we are 2 full days past string freeze, and we already made a mess with it, do we really need to uplift at this point?

If legal is fine with having logos (I assume it wasn't, hence my previous comment), and we don't risk to make last minute changes to pull them, my suggestion would be to leave this on mozilla-central and let the changes ride the train (it seems there are going to be more any way).

Comment 40

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/fc898522b3fb

Comment 41

[Ritu Kothari (:ritu) (Inactive, please n-i to RyanVM, jcristau, or pascal)]

Comment on attachment 8681192 [details] [diff] [review]
0001-Bug-1199354-add-link-to-Firefox-for-iOS-and-fix-link.patch

Approved for uplift to Aurora given that this is still early in the Aurora cycle. I have pinged Flod as well to know this makes sense.

Comment 42

[Francesco Lodolo [:flod]]

(In reply to Ritu Kothari (:ritu) from comment #41)
> Approved for uplift to Aurora given that this is still early in the Aurora
> cycle. I have pinged Flod as well to know this makes sense.

I guess it's OK. Comment 39 was based on the assumption that we already had a reference to iOS, while we don't in Aurora.

Comment 43

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-aurora/rev/9d368ecbd170

Comment 44

[Carsten Book [:Tomcat]]

https://hg.mozilla.org/releases/mozilla-b2g44_v2_5/rev/9d368ecbd170

Comment 45

[Camelia Badau [:cbadau], Desktop QA]

Verified fixed on Windows 7 64bit, Ubuntu 13.10 32bit and Mac OS X 10.9.5 using Nightyl 45.0a1 (buildID: 20151120030227 - for Windows and buildID: 20151122030230 - for Mac and Ubuntu) and latest Aurora 44.0a2 (buildID: 20151122004109).