All users were logged out of Bugzilla on October 13th, 2018

IME switching menu showing wrong current keyboard

VERIFIED FIXED

Status

VERIFIED FIXED
4 years ago
4 years ago

People

(Reporter: mnjul, Assigned: janjongboom)

Tracking

({regression})

unspecified
ARM
Gonk (Firefox OS)
regression
Bug Flags:
in-testsuite ?

Firefox Tracking Flags

(blocking-b2g:2.2+, b2g-v2.2 verified)

Details

Attachments

(1 attachment)

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.
Reverting bug 1048228 seems to fix the issue. Jan, could you take a look? Thanks!
Blocks: 1048228
blocking-b2g: --- → 2.2?
Flags: needinfo?(janjongboom)
Keywords: regression
(Assignee)

Updated

4 years ago
Assignee: nobody → janjongboom
Flags: needinfo?(janjongboom)
Created attachment 8502434 [details] [review]
Patch

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 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+
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
Last Resolved: 4 years ago
Resolution: --- → FIXED
set in‑testsuite? to indicate we would need an integration test to cover this.
status-b2g-v2.2: --- → fixed
Flags: in-testsuite?
Brogan, please verify that this issue is fixed.
Flags: needinfo?(bzumwalt)
Keywords: verifyme
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?]
status-b2g-v2.2: fixed → verified
Flags: needinfo?(bzumwalt) → needinfo?(ktucker)
Keywords: verifyme
QA Whiteboard: [QAnalyst-Triage?] → [QAnalyst-Triage+]
Flags: needinfo?(ktucker)
Blocking 2.2+ for all fixed regressions.
blocking-b2g: 2.2? → 2.2+
You need to log in before you can comment on or make changes to this bug.