Please refer to bug 816869. For supporting 3rd-party keyboards, we should need a new design of keyboard setting page in Settings App. I have a proposal. We could simply list all keyboards(input methods) installed, and user could enable/disable all of them. It looks like the "The UI design of Keyboard setting" picture in the attachment.

Hi Evan, I've responded to the email main thread with an update on the original proposal from Neo. He and I will be discussing this week in London and then follow up on this.

Hi Josh, I got it. Thanks. :)

Josh, any news on this?

Reassigning to Neo as he's taking care of 3rd party IME.

Update bug title to express engineering work here.

We've got a (draft) design from UX team as a reference for this bug. @Neo, Please update here for more details.

Comment on attachment 775320 [details] Latest UX design of IME. see Page 6 obsolete here because there is a copy in bug 891678 (meta bug), I will keep update there.

v1.2 feature. Nominate for koi+.

Evelyn, the commit is WIP v1. Since this is only a WIP, I did not add detail comments yet. I described the code briefly in github. Would you mind take a look on the overall design? Thanks!

Neo, could you help provide inputs on how to handle the case of users disabling all keyborad layouts?

This is the pull request to add 3rdParty keyboard support. Please be informed this change would need Gecko m-c to test with. For this bug, please help review bd22f6e0b76d1e5a4c4551f4082db4fff79ca871 only, the other 2 commits will be reviewed by other module peers/owners. The patch also covers bug 893557. Evelyn, could you help review it? Thanks!

I've tested on the phone, and I need more time to review the whole bunch of the code.

The other 2 parts of IME framework, i.e. keyboard app and keyboard manager have been r+'ed. So this is last part that we need to get the 1st phase implementation into Gaia master. Evelyn, Can you help to get to the review soon or we need to find someone else, like Kaze to help? Thank you.

I'm reviewing and discussing with Arthur. Should be done today.

Comment on attachment 790126 [details] link to https://github.com/mozilla-b2g/gaia/pull/11529 r=me. Arthur, I have to say: good job! Thanks for the big patch. As we discussed, the patch is an experiment to demonstrate how MVC/MVVM pattern can be applied in Settings app. I hope we can make our point more clear by separating it to two commits, refactoring the code structure, and filing another bug for discussion. Please address my commented on Github and update your commits, then flag a review to me for a check. Thank you so much. :)

Comment on attachment 790126 [details] link to https://github.com/mozilla-b2g/gaia/pull/11529 Evelyn, thank you for reviewing such a huge patch. And the suggestions are really helpful! Comment addressed as requested. As for a separate commit for the mvvm part, since the patch is combined with commits from Rudy and Gary, we will squash the commits before merge to master. I will make sure the mmvm part merged as a separate commit.

Comment on attachment 790126 [details] link to https://github.com/mozilla-b2g/gaia/pull/11529 r=me. Thanks.

Landed to Gaia master, bc8c1c962addd419b48ba12c6d8ddabdf9805391

backed out as part of the general backout of the 3rd party keyboard customizations (https://github.com/mozilla-b2g/gaia/commit/42aa1e2ccd22e464bffe6b22ee1182cece5dc2dc) Due to b2g-desktop failures (fail to start) https://travis-ci.org/mozilla-b2g/gaia/builds/10571231#L129.

master: https://github.com/mozilla-b2g/gaia/commit/f283bf33f37cad5d8c337b43766f4ec53eb406c2

Something in this gaia.json auto-commit: https://hg.mozilla.org/integration/b2g-inbound/rev/31ca106e662c ...broke the gaia unit tests, eg: https://tbpl.mozilla.org/php/getParsedLog.php?id=27215367&tree=B2g-Inbound https://tbpl.mozilla.org/php/getParsedLog.php?id=27214598&tree=B2g-Inbound The auto-commit corresponds to: https://hg.mozilla.org/integration/gaia-central/pushloghtml?fromchange=c258483698afbf2c2b7f27f8fa16922d0ee48880&tochange=75e0fbd934e2c835db9ecd10ec6b2900b357d53c So this and the others in that push have been backed out: https://github.com/mozilla-b2g/gaia/commit/c2f23f132accb38af4a46f9171bc6be25fdb99a1 https://github.com/mozilla-b2g/gaia/commit/9a8098e8a6b86a32da0eef08e507ab689435dfee https://github.com/mozilla-b2g/gaia/commit/884d0af45bc605c8e943e3080b2a909461795953 https://github.com/mozilla-b2g/gaia/commit/6a653be7c8b34f263342ead2037b4f8f3c2aff0e https://github.com/mozilla-b2g/gaia/commit/cfcb28b5069054c1ad631c5bf4ed9c2a94e06d8b Please run the gaia unit tests before pushing if possible! :-)

master: https://github.com/mozilla-b2g/gaia/commit/08901c788a3a4bd3efda5853d7ee9c8f6d04cf50