Closed
Bug 1144433
Opened 10 years ago
Closed 10 years ago
[Contacts] Cold launching the app shows unlocalized text for a brief second
Categories
(Firefox OS Graveyard :: Gaia::Contacts, defect)
Tracking
(blocking-b2g:2.2+, b2g-v2.2 verified, b2g-master verified)
People
(Reporter: pcheng, Assigned: ferjm)
References
Details
(Keywords: regression, Whiteboard: [3.0-Daily-Testing])
Attachments
(2 files)
106.79 KB,
text/plain
|
Details | |
46 bytes,
text/x-github-pull-request
|
arcturus
:
review+
bajaj
:
approval-gaia-v2.2+
|
Details | Review |
Description:
This issue occurs in Accented English. I cannot test this in French or other supported languages due to bug 1142758 present in current 3.0 master - the app doesn't get launched in the right locale/language.
STR:
0) Device is in Accented English
1) Cold launch Contacts app
Expected: Header and the rest of text display in Accented English upon launching
Actual: Header and other text display in English for a second before they become Accented. See video:
https://www.youtube.com/watch?v=K1flKwO97zQ
Repro rate: 5/5
Also attaching a logcat.
Device: Flame 3.0 Master (full flash 319MB mem)
BuildID: 20150317073344
Gaia: 738987bd80b0ddb4ccf853855388c2627e19dcc1
Gecko: 008b3f65a7e0
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 39.0a1 (3.0 Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:39.0) Gecko/39.0 Firefox/39.0
Reporter | ||
Comment 1•10 years ago
|
||
This issue does NOT occur on 2.2. Cold launching the app immediately shows all text in the designated language. Tested in Accented English, French, and German.
Device: Flame 2.2
BuildID: 20150317002504
Gaia: d0e09d5e6367e558824f9cbf691da99cedf63037
Gecko: 793d61bb0bd4
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
QA Whiteboard: [QAnalyst-Triage?]
status-b2g-v2.2:
--- → unaffected
status-b2g-master:
--- → affected
Flags: needinfo?(ktucker)
Keywords: regression
Whiteboard: [3.0-Daily-Testing]
Assignee | ||
Comment 2•10 years ago
|
||
This is probably because we delayed the lazy load of l10n on bug 1112551.
Assignee | ||
Updated•10 years ago
|
Assignee: nobody → ferjmoreno
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Comment 3•10 years ago
|
||
[Blocking Requested - why for this release]:
Nominating this 3.0? since this a regression from 2.2.
blocking-b2g: --- → 3.0?
Comment 4•10 years ago
|
||
Assignee | ||
Updated•10 years ago
|
Attachment #8580655 -
Flags: review?(francisco)
Comment 5•10 years ago
|
||
Comment on attachment 8580655 [details] [review]
[gaia] ferjm:bug1144433.contacts.l10n > mozilla-b2g:master
LGTM,
I guess if the cache patch goes to 2.2 this will need to be nominated to isnt?
Attachment #8580655 -
Flags: review?(francisco) → review+
Updated•10 years ago
|
Keywords: checkin-needed
Updated•10 years ago
|
Keywords: checkin-needed
Comment 6•10 years ago
|
||
Pull request has landed in master: https://github.com/mozilla-b2g/gaia/commit/9c4521cb9108c5169965f15d81900bf40e814154
Updated•10 years ago
|
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Assignee | ||
Comment 7•10 years ago
|
||
Comment on attachment 8580655 [details] [review]
[gaia] ferjm:bug1144433.contacts.l10n > mozilla-b2g:master
[Approval Request Comment]
[Bug caused by] (feature/regressing bug #): Bug 1112551. If that one goes to 2.2, we need to take this fix as well.
[User impact] if declined: Awful UX. The user will see unlocalized content for a second.
[Testing completed]: Manual test.
[Risk to taking this patch] (and alternatives if risky): Low risk, we just cache the localized strings and show them before we load l10n.
[String changes made]: None
Attachment #8580655 -
Flags: approval-gaia-v2.2?
Updated•10 years ago
|
blocking-b2g: 3.0? → 2.2+
Updated•10 years ago
|
Attachment #8580655 -
Flags: approval-gaia-v2.2? → approval-gaia-v2.2+
Assignee | ||
Updated•10 years ago
|
Comment 8•10 years ago
|
||
Target Milestone: --- → 2.2 S8 (20mar)
Comment 9•10 years ago
|
||
Unable to verify this bug as it is blocked by Bug 1144838. The "Contacts" header is not accented at all on Flame Master.
Device: Flame 3.0 (KK, 319mb, full flash)
Build ID: 20150402063750
Gaia: f37be8b44cb7c3a147b9615ab76743b760f08eeb
Gecko: 35046df9df1f
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (Master)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Reporter | ||
Comment 10•10 years ago
|
||
This issue is verified fixed on Flame 3.0 and 2.2. Cold launching Contacts app does not briefly show unlocalized text. Tested in Accented English, French, and German.
Device: Flame 3.0
BuildID: 20150420010204
Gaia: cb41d8421da5dc4f16ea566ea2917a9b7f828154
Gecko: 50b95032152c
Gonk: b83fc73de7b64594cd74b33e498bf08332b5d87b
Version: 40.0a1 (3.0)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:40.0) Gecko/40.0 Firefox/40.0
Device: Flame 2.2
BuildID: 20150420002502
Gaia: c15a2b6d3a783813959c2b3bffd2a131f4270b9e
Gecko: cc02ee38b252
Gonk: ebad7da532429a6f5efadc00bf6ad8a41288a429
Version: 37.0 (2.2)
Firmware Version: v18D-1
User Agent: Mozilla/5.0 (Mobile; rv:37.0) Gecko/37.0 Firefox/37.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage+] → [QAnalyst-Triage?]
Flags: needinfo?(ktucker)
Updated•10 years ago
|
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
You need to log in
before you can comment on or make changes to this bug.
Description
•