Closed Bug 1026328 Opened 10 years ago Closed 10 years ago

[OPENC_1.3]No 'Sound' and 'Alert' selected in settings app any more after changing language.

Categories

(Firefox OS Graveyard :: Gaia::Settings, defect, P1)

Product:
Firefox OS Graveyard
Component:
Gaia::Settings
Platform:
ARM
Gonk (Firefox OS)
Type:
defect

Tracking

(b2g-v1.3 affected, b2g-v2.0 affected, b2g-v2.1 fixed)

Status:
RESOLVED FIXED
Tracking Flags:
Tracking Status
b2g-v1.3 --- affected
b2g-v2.0 --- affected
b2g-v2.1 --- fixed

People

(Reporter: zhang.ruili, Assigned: dkuo)

Details

Attachments

(4 files)

English sound and alarm tones selected.png
19.02 KB, image/png
Details
Changed to French no sound or alarm selected.png
20.29 KB, image/png
Details
patch
46 bytes, text/x-github-pull-request
eragonj
: review+
Details | Review
VIDEO0155_Compress.MP4
6.03 MB, video/mp4
Details 
User Agent: Mozilla/5.0 (Windows NT 5.1) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/35.0.1916.153 Safari/537.36

Steps to reproduce:

1. Launch settings app and select “sound”.
2. Change 'Ringer', for example, to 'Classic Prism' and then tap done. Also, change 'Alert' sound.
3. Open 'Language' and change language to 'French' or any other language.
4. Reopen 'Sound' and 'Ringer' menu.

You'll find no any sound item selected, and so is 'Alert'.
Severity: normal → critical
blocking-b2g: --- → 1.3?
OS: All → Gonk (Firefox OS)
Priority: -- → P1
Hardware: All → ARM
Flags: needinfo?(vchen)
This issue has been reproduced with 1.3 buri (06/16) build:
Gecko-1da5885
Gaia-7f361e6

And also, with master (06/17) buri build:
Gecko-82463f1
Gaia-f93e6fa
Status: UNCONFIRMED → NEW
Ever confirmed: true
De-nom since it is not a critical bug to block 1.3.

Hi Howie, could you find someone to check this setting problem? It can be reproduced on master as well

Thanks
Flags: needinfo?(vchen) → needinfo?(hochang)
I would like to nominate the issue to be fixed in 2.0. Agree that we are late for 1.3.
blocking-b2g: 1.3? → 2.0?
status-b2g-v1.3: --- → affected
status-b2g-v2.0: --- → affected
status-b2g-v2.1: --- → affected
This is something we should fix, but not a blocker, ni Dominic, can you help on this? Thanks.
blocking-b2g: 2.0? → ---
Flags: needinfo?(hochang) → needinfo?(dkuo)
Attached file patch — 
EJ,

This issue is a lack of the ringtones work from 2.0, the patch is trivial that added the missing l10nId, would you please review it? thanks.
Attachment #8444240 - Flags: review?(ejchen)
Flags: needinfo?(dkuo)
Hey EJ,

Thanks for finding the issue that we found in the offline discussion, I have addressed it and now the custom ringtone should not be localized but keep as it is, would you review it again? thanks.
Flags: needinfo?(ejchen)
Domi, just left a comment on Github and please check it ! Thanks :)
Flags: needinfo?(ejchen)
Thanks EJ and I have addressed the issue, mind to take a look again? thanks.
Flags: needinfo?(ejchen)
Comment on attachment 8444240 [details] [review]
patch

It looks nice ! r+

Thanks for the help, Domi !
Attachment #8444240 - Flags: review?(ejchen) → review+
master: 9007d3846dd46e04b1ef8eed92bc482260206511
Status: NEW → RESOLVED
Closed: 10 years ago
status-b2g-v2.1: affected → fixed
Resolution: --- → FIXED
Assignee: nobody → dkuo
Tested with unagi master and working OK. 

Gecko-0112de7
Gaia-512f288
Attached video VIDEO0155_Compress.MP4 — 
This issue has been successfully verified on Flame 2.1:
Gaia-Rev        38e17b0219cbc50a4ad6f51101898f89e513a552
Gecko-Rev       https://hg.mozilla.org/releases/mozilla-b2g34_v2_1/rev/8b92c4b8f59a
Build-ID        20141205001201
Version         34.0
Device-Name     flame
FW-Release      4.4.2
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: