Closed Bug 1593320 Opened 5 years ago Closed 4 years ago

Fennec/Fenix do not bring up virtual keyboard a testcase, Chrome does

Categories

(GeckoView :: IME, defect, P2)

Unspecified
Android
defect

Tracking

(Webcompat Priority:revisit, firefox71 affected)

RESOLVED DUPLICATE of bug 1658948
Future
Webcompat Priority revisit
Tracking Status
firefox71 --- affected

People

(Reporter: twisniewski, Unassigned)

References

Details

Attachments

(1 file)

Attached file testcase.html

In the attached testcase, Chrome still brings up the virtual keyboard when tapping on the overlay covering the textarea, despite the mousedown event being default-prevented. Firefox does not.

Chrome seems to be bringing up the keyboard because the textarea is already focused, regardless of whether the muosedown is canceled (if I remove the focus() call from the testcase, Chrome also eats the event without bringing up the keyboard).

Note that I'm not sure if the same behavior will be present on desktop editions of the browser with touchscreens and virtual keyboards, so it might be Android-specific.

Whether Chrome's behavior is reasonable or not, it may be desirable to users, as per the related webcompat issue.

(not sure about the right component. but at least not DOM Events)

Component: DOM: Events → General
Product: Core → GeckoView
Target Milestone: --- → Future

This could be a GV IME issue.

Rank: 44
OS: Unspecified → Android
Priority: -- → P2

Hmm, our implementation is that nsIWidget::SetInputContext shows VKB. But SetInputContext isn't called even if tapping textarea on sample HTML. So I guess that this may occur on Windows's VKB for tablet mode.

Isn't nsFocusManager::Focus called on this HTML?

Nakano-san, do we have any change that IMEStateManater calls SetInputContext on this test case?

Flags: needinfo?(masayuki)

Hmm, I have no idea. It should be called from EditorEventListener::MouseClick() via IMEStateManager::OnClickInEditor(). I guess the changes for Fission must not affect this path.

Flags: needinfo?(masayuki)
Webcompat Priority: ? → revisit

I am working similar issue on another bug.

Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → DUPLICATE

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

Component: General → IME
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: