Closed
Bug 1080357
Opened 9 years ago
Closed 9 years ago
IME switching menu showing wrong current keyboard
Categories
(Firefox OS Graveyard :: Gaia::System::Input Mgmt, defect)
Tracking
(blocking-b2g:2.2+, b2g-v2.2 verified)
People
(Reporter: mnjul, Assigned: janjongboom)
References
Details
(Keywords: regression)
Attachments
(1 file)
STR: 1. Enable a few keyboards, such as English, French, Spanish, Polish 2. Open an app and focus a textbox to bring p the keyboard. Suppose you're then at English keyboard 3. Switch to some other keyboard, say Spanish 4a. Pull down utility tray, click the IME switcher entry ("Built-in Keyboard: Español, Tap to switch input method") to bring up the IME switching menu, OR 4b. Long press the short label of the keyboard layout to bring up the IME switching menu Actual results: At step 4a or 4b, the IME switching menu indicates that English is the current keyboard, which is wrong. Expected results: At step 4a or 4b, the IME switching menu indicates the current keyboard correctly as Español. Note: the IME switcher in the utility tray itself shows the current keyboard correctly. It's the menu that it launches that's wrong.
Reporter | ||
Comment 1•9 years ago
|
||
Reverting bug 1048228 seems to fix the issue. Jan, could you take a look? Thanks!
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → janjongboom
Flags: needinfo?(janjongboom)
Assignee | ||
Comment 2•9 years ago
|
||
The problem here is that we save all settings in keyboard_helper whenever someone changes the current layout. This triggers settings listeners for layouts (for when you add new layout through Settings app), which resets the activeLayout to 0 for all groups (in input_layouts.js). The ImeMenu looks at that, sees 0 and selects English from there. This patch only updates the activelayout setting to get around that.
Attachment #8502434 -
Flags: review?(timdream)
Comment 3•9 years ago
|
||
Comment on attachment 8502434 [details] [review] Patch Looking at this patch alone I wonder if we need to add some tests on this part.
Attachment #8502434 -
Flags: review?(timdream) → review+
Assignee | ||
Comment 4•9 years ago
|
||
It would need an integration test I think. Unit testing these event wired things is a pita. Try is green https://treeherder.mozilla.org/ui/#/jobs?repo=gaia-try&revision=4166e4874940 Landed https://github.com/mozilla-b2g/gaia/commit/61a874ba48614f9e73cba4d758aac1451eaada04
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Comment 5•9 years ago
|
||
set in‑testsuite? to indicate we would need an integration test to cover this.
status-b2g-v2.2:
--- → fixed
Flags: in-testsuite?
Comment 6•9 years ago
|
||
Brogan, please verify that this issue is fixed.
Flags: needinfo?(bzumwalt)
Keywords: verifyme
Comment 7•9 years ago
|
||
Issue verified fixed on Flame 2.2 Actual results: IME switching menus display active keyboard language correctly Device: Flame 2.2 Master (319mb)(Kitkat Base)(Full Flash) BuildID: 20141024040202 Gaia: d893a9b971a0f3ee48e5a57dca516837d92cf52b Gecko: a5ee2769eb27 Gonk: 05aa7b98d3f891b334031dc710d48d0d6b82ec1d Version: 36.0a1 (2.2 Master) Firmware: V188 User Agent: Mozilla/5.0 (Mobile; rv:36.0) Gecko/36.0 Firefox/36.0
Status: RESOLVED → VERIFIED
QA Whiteboard: [QAnalyst-Triage?]
Flags: needinfo?(bzumwalt) → needinfo?(ktucker)
Keywords: verifyme
Updated•9 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
•