1022856 - implement translation provider attribution

Status: VERIFIED FIXED

(Firefox :: Translations, defect)

Description

[:Gavin Sharp [email: [e-mail]]]

The translation provider (Bing) has attribution requirements for us of their API. We're still sorting out what those are exactly, but we'll need to add a logo/wording before launching the trial.

Comment 1

[Florian Quèze [:florian]]

(In reply to :Gavin Sharp (email [e-mail]) from comment #0)

> we'll need to add a logo/wording before launching the trial.

Is this 'wording' something that will require new localizable strings?

Comment 2

[:Gavin Sharp [email: [e-mail]]]

Unknown yet, but if it does we can live with non-localized strings for the trial.

Comment 3

[Marco Mucci [:MarcoM]]

Hi Felipe, are you taking this bug this iteration?

Comment 4

[:Felipe Gomes (needinfo for replies!)]

Attachment: Provider attribution, WIP

Almost there..  Took me a while fiddling with the classes to get the right footer style, but luckily the Australis work had implemented footer styles for menupopups too, not just panels.  So I was able to use them.

It needs a bit more fiddling with the right sizing, and positioning in the Preferences dialog. Note that the most important now is only the old-style Preferences window, because in-content prefs have been backed-out from Aurora.

Also need to replace the image with one that includes the "Translations by" text.

I've tested it on Mac, Win7, Win Basic, Win Classic..  Only needs more work in high-contrast mode, but that should be left for another bug.

Comment 5

[:Felipe Gomes (needinfo for replies!)]

Marco: yeah, but me and Florian will be working together on this bug. I'll provide the point estimate and QA flag tomorrow after talking with him.

Comment 6

[Marco Mucci [:MarcoM]]

Added to Iteration 33.1

Comment 7

[:Felipe Gomes (needinfo for replies!)]

Attachment: microsoft-translation-attribution.png 140px wide

Comment 8

[Florian Quèze [:florian]]

Attachment: v2

Comment 9

[:Felipe Gomes (needinfo for replies!)]

Attachment: Final patch

Ran the image through pngcrush, changed text to German and adjusted a small detail for the Windows' CSS

Comment 10

[Florian Quèze [:florian]]

Comment on attachment 8443649 [details] [diff] [review]
Final patch

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

r=me for the parts Felipe wrote.

Comment 11

[:Felipe Gomes (needinfo for replies!)]

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

Comment 12

[Marco Mucci [:MarcoM]]

Thanks for the update Felipe.  We'll be transitioning to the new fields starting with iteration 33.2.

Comment 13

[Florian Quèze [:florian]]

Taking to match the author in the changeset Felipe pushed, but we've really both done half the work here.

Comment 14

[:Felipe Gomes (needinfo for replies!)]

Comment on attachment 8443649 [details] [diff] [review]
Final patch

[Approval Request Comment]
Bug caused by (feature/regressing bug #): This bug is part of the automatic translation feature, which we want to A/B with a subset of Aurora 32 users.
User impact if declined: The feature won't have the service attribution required
Testing completed (on m-c, etc.): landed on inbound
Risk to taking this patch (and alternatives if risky): small, all code is only exposed to users who have the feature enabled (which is off by default)

String or IDL/UUID changes made by this patch: There's a hardcoded string that says "Translations by" in German. I talked with Gavin/Sevaan and we agreed that it's an acceptable trade-off for now as the experiment will only run for german users, and the design/string may not be final. Once it's finalized we will remove the hardcoded string and land it on central for translation.

This string is directly in the code so it won't cause any problems for the l10n tools

Comment 15

[Lawrence Mandel [:lmandel] (use needinfo)]

Comment on attachment 8443649 [details] [diff] [review]
Final patch

I'm granting Aurora approval before this lands on m-c (as is the usual process) in order to support the translation A/B test as Felipe has requested. Although this code introduces a hard coded German string, as Felipe noted, the test is only for German and this code will not hit release before it is fixed (on m-c).

Felipe:
1. Is your intention to back out this code after the test or leave it disabled on 32 as it rides to beta and release?
2. As with your other recent bugs, please handle the Aurora landing as there is no sheriff support until next week.

Comment 16

[:Felipe Gomes (needinfo for replies!)]

Thanks Lawrence. I built Aurora with the patch, made sure the feature works and ran its tests. I'll also be keeping an eye on Aurora and will backout if any problem happens.

> 1. Is your intention to back out this code after the test or leave it disabled on 32 as it rides to beta and release?

No backout, we will leave it disabled and when it reaches beta we will be activating this code again through the second trial (for beta users of 3 locales). There's no intention on enabling this feature for release users in the near future.

As for the string, the idea is to remove the hardcoded version from the .js file on central and land it properly on a localizable file when we're committed to the string. And we'll see how to handle it for the 3 locales chosen for the beta trial, but that is the subject for a later bug.

Comment 17

[:Felipe Gomes (needinfo for replies!)]

https://hg.mozilla.org/releases/mozilla-aurora/rev/8169edbeb8f7

Comment 18

[Tim Taubert [:ttaubert] (inactive)]

https://hg.mozilla.org/mozilla-central/rev/2d9116d8711b

Comment 19

[Marco Mucci [:MarcoM]]

Hi Florin, can a contact be assigned for QA verification.

Comment 20

[Bogdan Maris, Desktop Test Engineering]

Tested on Windows 7 64bit, Windows 8.1 64bit Surface Pro 2, Mac OS X 10.9.3 and Ubuntu 14.04 32bit using latest Nightly and latest Aurora. Verified that the translation provider icon is displayed in Translation infobar/Options, about:preferences and Options menu (Aurora).

Comment 21

[Florian Quèze [:florian]]

Attachment: Screenshot of what was implemented