Closed Bug 981604 Opened 11 years ago Closed 11 years ago

[System]: Time Picker settings (12h/24h, reel order) are not updated when language changes

Categories

(Firefox OS Graveyard :: Gaia::System, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: flod, Assigned: flod)

References

Details

(Keywords: l12y, Whiteboard: [lang-switching])

Attachments

(1 file)

(deleted), text/x-github-pull-request
kaze
: review+
Details
I noticed this when doing tests for bug 932356. Apparently there's code in value_selector.js to reset the Date Picker when language changes, but not for Time Picker. I think I understand have to fix this issue, replicating what's been done for Date Picker, going to take a better look in the next days.
Attached file Github PR (deleted) —
Not urgent, so I'll leave this for :kaze when he's back. There are two problems in the current code: * We need to "uninit" the Time Picker when language changes, but just setting it to "not initialized" is not enough. Format changes, but reels act funny (wrong positions, am/pm double). Not sure if what I'm resetting is enough. * We add a class to picker-container, which is not good (classes pile up at every language change). Patch works fine on my Keon.
Attachment #8388543 - Flags: review?(kaze)
Note to reproduce: you need to open time picker at least once in the previous language. Example: 1. Open time picker with phone set to English 2. Switch to French 3. Open time picker, it will still use 12h format
Summary: [System]: Time Picker doesn't change when language is changed in Settings → [System]: Time Picker settings (12h/24h, reel order) are not updated when language changes
Keywords: l12y
Whiteboard: [lang-switching]
Comment on attachment 8388543 [details] Github PR LGTM
Attachment #8388543 - Flags: review?(kaze) → review+
Thanks Kaze, I should be done for a while with review requests ;-)
Keywords: checkin-needed
Status: NEW → RESOLVED
Closed: 11 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Verified on Keon, Commit Git 2014-03-24 b9fffe49
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: