1213502 - Marketing][screenshots][de]: FF for Android 42

Status: RESOLVED FIXED

Description

[Peiying Mo [:CocoMo]]

Hi Sebastian,

This is very much like bug 1206949 but for Android.  We have leveraged the work you did so you only need to provide the translation for new strings. (You have worked on it earlier on Friday but the content was revised. What's shown are new)

*Source file*: https://github.com/mozilla-l10n/appstores/blob/master/de/android_42_release.lang 
*Project progress page*: https://l10n.mozilla-community.org/~pascalc/stores_l10n/locale/de/google/next/
*Locale Project Dashboard*: https://l10n.mozilla-community.org/webdashboard/?locale=de

Try to finish localizing the strings ASAP so Tannie can start his part his part.

*Deadline*: The completed localized screenshots are due October 30th.

Comment 1

[Michael Köhler [:Coce]]

Done.
https://github.com/mozilla-l10n/appstores/blob/master/de/android_42_release.lang

Comment 2

[Peiying Mo [:CocoMo]]

Thank you!

We need Tannie to make the screenshots.

Comment 3

[Tannie Low]

It doesn't seem like all the strings are there: https://l10n.mozilla-community.org/~pascalc/stores_l10n/locale/de/google/next/

Comment 4

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

Hi Tannie, they have been translated. You can find them at https://github.com/mozilla-l10n/appstores/blob/master/de/apple_description_release.lang - no idea why https://l10n.mozilla-community.org/~pascalc/stores_l10n/locale/de/google/next/ didn't pick them up. Pascal, can you check what's going on here, please?

Comment 5

[Peiying Mo [:CocoMo]]

Hello Sebastian, please find the "de" folder and review the screenshots: https://app.box.com/s/mnungopxcfdu7km8noxmcim48vlg5qbj.  Thanks!

Comment 6

[Michael Köhler [:Coce]]

If I may, I just have a few comments:

> FFx-Android_Nexus7_DE-Tabs.png
All pages are English.

> FFx-Android_Nexus7_DE-Sync.png
I am not aware of what Firefox for Android looks like, is it correct that the favicon and the HTTPS lock are both in the URL bar?

> FFx-Android_Nexus7_DE-Search.png
Only English pages are being displayed.

> FFx-Android_Nexus7_DE-QuickShare.png
The page is English, and as a follow-up to the comment above, there is no favicon.

> FFx-Android_Nexus7_DE-Privacy.png
The text in the URL bar is German, the rest is English, including the tab’s title.


> FFx-Android_Nexus5_DE-Tabs.png
Only English pages are shown. At the very bottom I again notice the favicon and the lock.

> FFx-Android_Nexus5_DE-Sync.png
Favicon + lock.

> FFx-Android_Nexus5_DE-Search.png
Only English pages are being displayed

> FFx-Android_Nexus5_DE-QuickShare.png
Favicon + lock.

> FFx-Android_Nexus5_DE-Privacy.png
The text in the URL bar is German, the rest is English.

> FFx-Android_Nexus5_DE-Add-Ons.png
Favicon + lock.


> FFx-Android_Nexus10_DE-Tabs.png
Only English pages are shown, but in this case it seems so small to me, that it’s barely an issue (imho).

> FFx-Android_Nexus10_DE-Sync.png
Favicon + lock.


> FFx-Android_Nexus10_DE-Search.png
Only English pages are visible.

> FFx-Android_Nexus10_DE-QuickShare.png
On this occasion, there is no favicon.

> FFx-Android_Nexus10_DE-Privacy.png
The text in the URL bar is German, most of the rest is English.

Comment 7

[Tannie Low]

Many or all of the in-product screenshots were taken from an actual device. You'll see that much of the content may be in English - however the hope was than many of the buttons and other parts of the UI would be localized.

With this in mind, please only check the accuracy of the screens and not necessarily their percentage of localization. Thanks!!

Comment 8

[Tannie Low]

In which case, it seems like all your comments fall in line with the information just covered. 

So, I think we're good to go with this set!

Comment 9

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

Unfortunately, we missed a missing comma in these kind of screenshots: https://app.box.com/s/mnungopxcfdu7km8noxmcim48vlg5qbj/1/5086860894/41171164610/1 This also affects iOS and many people likely wouldn't notice it. The required change is here: https://github.com/mozilla-l10n/appstores/commit/ca97aa70438420d096bba2055ab97d6905d9a497 Sorry for the inconvenience.

Comment 10

[Tannie Low]

Images updated for review: https://app.box.com/s/zsx2jvhitlils6axzi8tffxnm87dyq16

Comment 11

[Sebastian Hengst [:aryx] (needinfo on intermittent or backout)]

Thank you for the fast fix.