Closed Bug 1409252 Opened 7 years ago Closed 7 years ago

Firefox 57 for Android new Photon screenshot approval

Categories

(Mozilla Localizations :: Other, enhancement)

enhancement
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: mhan, Unassigned)

References

Details

Hi,

With the upcoming Firefox 57 for Android release, we will be updating the in product screenshots of our app store screenshots.

You can find the current app store screenshots here:

https://play.google.com/store/apps/details?id=org.mozilla.firefox

(Please ensure you are going to the correct page for your locale)

We are only replacing the in product screenshots, 1:1 with the existing.

Locales required for approval: zh-cn, RU, PT-PT, PT-BR, JA, IT, ID, FR, ES-US, DE

Deadline: 10/29/17

In product screenshot can be found here: 

https://drive.google.com/drive/folders/0B8pVQseE4Eg3MktQSkQzUHJZZEU
All the "privacy" images mention Firefox Beta in the text, I don't think that's what you want/expect?
Thanks - I'll get bugs on file for each locale shortly. In the meantime, could you please confirm about the Firefox Beta in the text for the Pricacy images - and provide an updated version if needed?
(also, does es-US correspond to our es-MX (Mexican Spanish)? I assume so, but want to make extra sure so this goes out to the right team to review ;) )
Flags: needinfo?(mhan)
ahhh.. good catch.  i missed that one.  We'll update that slide without the "beta".  But Im not sure when we'll get that.  Can we go ahead with the review, and we'll make sure to capture the change request?   All of these screenshots are coming from text directly coming from the build.  

regarding es-US - its labeled as spanish in the US - how have we handled l10n in the past for this?
Flags: needinfo?(mhan)
Mike - I've filed the bugs for review and added them as attachments here. Localizers will close the bugs if screenshots are good to go. I'll check in with you after a few days if we have any corrections to make. thanks
Mike: es-ES and zh-TW are also in that google drive but not under your list of required locales that need to check. I assume we need them to sign-off as well? (going to file bugs for them in the meantime just in case)
Flags: needinfo?(mhan)
sorry i missed es-es.  These screenshots were created by the Taipe team, so they've already been looked at.  However, it doesnt hurt to get peters feedback!
Flags: needinfo?(mhan)
And by the way, the updated privacy screenshots with "beta" removed are already in the same folders.
Thanks Mike! 
I've already received Russian feedback in Bug 1409930 that there are changes made to the original text as well as one typo fixed. Not sure if you want to go ahead and have those screenshots generated again for Russian - maybe help to buy some time. Or we can sync-up next week while the other localizers are still reviewing. Whatever works best.
Hey Mike - answering here to https://bugzilla.mozilla.org/show_bug.cgi?id=1409939#c4:
What I meant by "changing" was that the Russian localizer went in and fixed a typo. I'm not sure on which screenshot, and if it's critical enough to justify an update, so I asked the localizer in the bug and we shall see what he answers.
We don't need to regenerate the New Tabs screenshot for all locales, sorry the confusion.
Given you're only looking for glaring errors - since teams are busy with Quantum and this is a manual process - I'll just list here next week any major issues we've encountered and that absolutely need to be fixed.
I'll start listing issues that are critical and absolutely a fix:
* Russian: sceenshot is Z3C_Privacy.png. There was a glaring typo there that needs to be udpated
Sorry, seems like I didn't explain in detail comment 10:
Error to be fixed in Russian:
* screenshot is Z3C_Privacy.png: https://pontoon.mozilla.org/ru/firefox-for-android/all-resources/?search=We+won%27t+remember+any+history%2C+but+downloaded+files+and+new+bookmarks+will+still+be+saved+to+your+device.&string=169699
That's the correct string to use above. Please update on screenshots. thanks!
Flags: needinfo?(mhan)
* es-US feedback: there are multiple screenshots here that are *entirely* in English (so it's not english + spanish text mixed together - it's english *only*).

Can we just use es-ES (so Spain Spanish) version of screenshots for es-US, since they are much better?
For ES-US, yes we can use es-es screenshots
Flags: needinfo?(mhan)
(In reply to Delphine Lebédel [:delphine - use Need Info] from comment #12)
> * es-US feedback: there are multiple screenshots here that are *entirely* in
> English (so it's not english + spanish text mixed together - it's english
> *only*).
> 
> Can we just use es-ES (so Spain Spanish) version of screenshots for es-US,
> since they are much better?

¡Gracias!
Hi Delphine,

Do we have any additional feedback on updates that need to be made or are all other locales approved?

Currently I have captured the updates to RU (copy) and ES-US (to use es-es).
Flags: needinfo?(lebedel.delphine)
Hey Mike - you're right that's all there is to update. The rest is signed-off (attached bugs are resolved-fixed). thanks
Flags: needinfo?(lebedel.delphine)
great!  thanks!
Closing this bug since all issues have been reported and will be acted upon - and the rest is signed-off on and good to go!
thanks everyone!
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.