1658948 - Software keyboard isn't shown again on contenteditable.

Status: RESOLVED FIXED

(GeckoView :: IME, defect, P1)

Description

[Makoto Kato [:m_kato]]

From https://github.com/mozilla-mobile/fenix/issues/11785

Steps to reproduce

1. Go to www.Onlinegdb.com/online_c++_compiler
2. click on the centre text box
3. keyboard appears
4. press back
5. click again on the centre text box

Expected behavior

Keyboard appears. Note that for most other text boxes this is the current behaviour

Actual behavior

Nothing.

Device information

Android device: Asus Zenfone
Fenix version: Nightly

Comment 1

[Makoto Kato [:m_kato]]

This site uses Ace editor (https://ace.c9.io/).

Comment 2

[Makoto Kato [:m_kato]]

https://jsfiddle.net/qhpwj1L9/4/

Ace doesn't use contentedtiable, it uses <textarea> to get input. Chrome opens software keyboard, but Firefox doesn't. I guess that this issue will occurs on Firefox for Windows and editor won't set InputContext on this situation.

Comment 4

[Makoto Kato [:m_kato]]

Attachment: Bug 1658948 - Ace editor doesn't show software keyboard. r=masayuki

Actually software keyboard is opened by the following situations.

• Click event in editor
• Get new focus event.

Ace editor has small <textarea> element to handles some input events.
When clicking editable-like area (but this isn't ediable on Ace editor),
mousedown handler calls event.preventDefault() then set focus to
<textarea> element. So editor doesn't get click event and focus is still
keep in <textarea> element. At finally, we cannot open software keyboard
again since focus isn't changed.

So even if focus isn't changed, we should call SetInputContext to open
software keyboard.

Also, actually we doesn't have generic test code for notification of
software keyboard change (bug 1670539). I will add the test by bug 1670539.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/7b96efde9ee1
Ace editor doesn't show software keyboard. r=masayuki

Comment 6

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/7b96efde9ee1

Comment 7

[Irvin (MozTW)]

Regressions: bug 1675313

Comment 8

[Chris Peterson [:cpeterson]]

Moving some keyboard bugs to the new GeckoView::IME component.