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)
Firefox OS Graveyard
Gaia::System
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: flod, Assigned: flod)
References
Details
(Keywords: l12y, Whiteboard: [lang-switching])
Attachments
(1 file)
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.
Assignee | ||
Comment 1•11 years ago
|
||
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)
Assignee | ||
Comment 2•11 years ago
|
||
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
Comment 3•11 years ago
|
||
Comment on attachment 8388543 [details]
Github PR
LGTM
Attachment #8388543 -
Flags: review?(kaze) → review+
Assignee | ||
Comment 4•11 years ago
|
||
Thanks Kaze, I should be done for a while with review requests ;-)
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Assignee | ||
Comment 6•11 years ago
|
||
Verified on Keon, Commit Git 2014-03-24 b9fffe49
Assignee | ||
Updated•11 years ago
|
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•