FTE : Tutorial needs to be translated for the graphics or have images that do not have english in it

VERIFIED FIXED in B2G C3 (12dec-1jan)

Status

Firefox OS
Gaia::First Time Experience
P1
normal
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: nhirata, Assigned: fcampo)

Tracking

({late-l10n})

unspecified
B2G C3 (12dec-1jan)
ARM
Gonk (Firefox OS)
late-l10n
Dependency tree / graph

Firefox Tracking Flags

(blocking-basecamp:+)

Details

(Whiteboard: [target:12/21])

Attachments

(1 attachment)

## Environment :
Unagi phone, build 2012-12-9

## Repro :
1. set language to a P1 language
2. run through the first run experience
3. launch the tutorial

## Expected :
1. tutorial should be in the selected language

## Actual :
1. English is shown

## Note :
1. seperating from bug 819960
Triage: BB+, P1 - tutorial in wrong lanaguage is not useful for users
blocking-basecamp: ? → +
Keywords: late-l10n
Priority: -- → P1
(Assignee)

Comment 2

6 years ago
Yup, discovered that even though strings are localizad, l10n tags are not used. 

Taking care of it together with #818588 and #813302
Assignee: nobody → fernando.campo
Whiteboard: [target:12/21]
(Assignee)

Comment 3

6 years ago
Created attachment 691868 [details]
Link to Will get fixed when https://github.com/mozilla-b2g/gaia/pull/6999 gets approved (asking for revisions to the patch on https://bugzilla.mozilla.org/show_bug.cgi?id=820150

This fixes the translations, but with the new assets provided by UX

Would solve at the same time bug 813302, bug 818350 and bug 818588 (all tutorial related)
Attachment #691868 - Flags: review?(l10n)
Attachment #691868 - Flags: review?(fbsc)

Comment 4

6 years ago
Comment on attachment 691868 [details]
Link to Will get fixed when https://github.com/mozilla-b2g/gaia/pull/6999 gets approved (asking for revisions to the patch on https://bugzilla.mozilla.org/show_bug.cgi?id=820150

From an l10n pov, I think we want to nail down the copy before landing this. I've tried to rally help in bug 818588.
Attachment #691868 - Flags: review?(l10n) → review-
Attachment #691868 - Flags: review?(fbsc) → review+
(Assignee)

Updated

6 years ago
Depends on: 818588
(Assignee)

Updated

6 years ago
Blocks: 813302
(Assignee)

Updated

6 years ago
Blocks: 818350

Updated

6 years ago
Target Milestone: --- → B2G C3 (12dec-1jan)
Blocks: 818588
No longer depends on: 818588
(Assignee)

Comment 5

6 years ago
Comment on attachment 691868 [details]
Link to Will get fixed when https://github.com/mozilla-b2g/gaia/pull/6999 gets approved (asking for revisions to the patch on https://bugzilla.mozilla.org/show_bug.cgi?id=820150

Final strings decided in bug 818588 (at last!), so we can keep going with the review :)
Attachment #691868 - Flags: review- → review?(l10n)

Comment 6

6 years ago
Comment on attachment 691868 [details]
Link to Will get fixed when https://github.com/mozilla-b2g/gaia/pull/6999 gets approved (asking for revisions to the patch on https://bugzilla.mozilla.org/show_bug.cgi?id=820150

r=me.

The only caveat I have would be a comment for RTL locales if "right" is "right" or "left", i.e., if we're ordering the screens in RTL or LTR for a RTL locale.

Given that we don't have an active RTL locale, and it'd be a comment only, that's not blocking, though.
Attachment #691868 - Flags: review?(l10n) → review+
(Assignee)

Comment 7

6 years ago
And merged! https://github.com/mozilla-b2g/gaia/commit/d85fdfc8cb9d70b264321e633a047f66dcbbcea2

About the RTL issue, I wouldn't worry for now, as we don't have either CSS rules for it (when I say not to worry, I mean that all the RTL issues will be probably brought up together in the future, when we are in disposition of taking care of them)
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
The issue with non-localized graphics is captured in bug 828276.

Localized text in tutorial confirmed fixed in build 20130110070201 using ES locale.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.