+++ This bug was initially created as a clone of Bug #1015643 +++ I made a mistake in the said bug where LoayoutLoader.initKeyboard(layoutName) only init the named layout. A file might come with multiple layouts so it must load all of it. STR: 1. enable Pinyin IME 2. Press ?123 key Expected: 1. Switch to the special Pinyin symbol layout Actual: 1. Do nothing another STR (need re-flash to enable Greek layout): 1. Switch to Greek 2. Go to Messsage app -> new message 3. Tap Shift key Expected: 1. Shift key should be disabled because we should be in all-caps ASCI-only fake Greek layout Actual: 1. I don't know yet but it should be something different.
Comment on attachment 8439734 [details] [review] mozilla-b2g:master PR#20479 I am thankful that we didn't land the previous bug on v2.0!
Attachment #8439734 - Flags: review?(rlu)
Target Milestone: --- → 2.0 S4 (20june)
Comment on attachment 8439734 [details] [review] mozilla-b2g:master PR#20479 Thanks for finding this issue, r+. -- Guess we could catch this earlier if we have the following integration tests, bub 949961: Integration tests for Pinyin IME & input method API composition methods bug 1024885: Add integration test to cover Greek SMS keyboard layout (just filed) Will try to work on those bugs as soon as possible.
Attachment #8439734 - Flags: review?(rlu) → review+
Status: ASSIGNED → RESOLVED
Last Resolved: 5 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.