Bug 43602 – [gtk] gtk_im_context_focus_in() should only be called when an input-able element has focus

• Status: RESOLVED
• Resolution: FIXED
• Component: WebKitGTK

[Brian Tarricone]

We have an on-screen keyboard that is implemented as a Gtk IM module.  The change in Martin's commit 14a3dc3801b495632521c557be1091e79b996b03 that adds a call to gtk_im_context_focus_in() in webkitwebview.cpp:webkit_web_view_focus_in_event() causes the keyboard to pop up all the time, whenever the web view is focused, even if there are no actual editable/input-able elements on the page.

Not sure what the best fix is; commenting out those lines makes the situation better, but there are still some problems.  I'm thinking it should be checking the type of the focused DOM node, and only enable it if it's something that can accept keyboard input.  But not sure how to do that just yet.

[Brian Tarricone]

Oh, that's svn rev r56072 for the git-impaired ^_^

[Martin Robinson]

Created attachment 63735 [details]
Patch for this issue

[Martin Robinson]

Thanks for reporting this Brian! I've uploaded a patch for this issue, but I do not think I can test it locally. Do you mind testing it for yourself? If it works, I'll mark it r?.

[Brian Tarricone]

Works perfectly, thanks Martin!

[Xan Lopez]

Comment on attachment 63735 [details]
Patch for this issue

r=me

[Martin Robinson]

Comment on attachment 63735 [details]
Patch for this issue

Clearing flags on attachment: 63735

Committed r64983: <http://trac.webkit.org/changeset/64983>

[Martin Robinson]

All reviewed patches have been landed.  Closing bug.