context menu steals focus from the webview

I fail to reproduce the issue. Tested on both krillin and arale running latest rc-proposed, also tested with oxide 1.11.3 from silo 57, pasting a link into gmail new message field always works for me.

seeing same behavior in twitter, where i can't paste into the New Tweet field

once in a while I can get the paste to work if I try 3 or 4 times, but not sure how to trigger it. Seems like it's related to focus of the field and perhaps the focus of the field is getting lost when the context menu is being displayed so it doesn't know where to paste the results. It does seem like the context menu is causing the osk to hide/show after selecting the paste option so guessing this is the cause

This (the fact that the context menu steals focus from the webview) can easily be observed in the browser by tapping on a text field to focus it (the OSK comes up), then long-pressing on that field to open the context menu: the OSK disappears, and doesn’t come back up when dismissed.

The context menu should never steals focus from the webview (stealing it and restoring it afterwards doesn’t sound good enough, as it would make the OSK hide/show again unnecessarily).

Unfortunately for the browser case (where the context menu is implemented as a modal Dialog), we’re out of luck because the implementation of Dialog explicitly grabs focus when becoming visible (see https://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/src/Ubuntu/Components/Popups/1.3/Dialog.qml#L147).

For the default context menu exposed by the Ubuntu WebView (which is what is used in e.g. the gmail webapp), the implementation uses an ActionSelectionPopover, which inherits Popover, which doesn’t steal focus when displayed. However tapping an action in the menu does steal the focus.

If I change the implementation of Popover (/usr/lib/arm-linux-gnueabihf/qt5/qml/Ubuntu/Components/Popups/1.3/Popover.qml) to not request active focus on press (https://bazaar.launchpad.net/~ubuntu-sdk-team/ubuntu-ui-toolkit/trunk/view/head:/src/Ubuntu/Components/Popups/1.3/Popover.qml#L209), then the webview retains focus as expected.

Applying the above workaround without modifying the UITK is easy enough with the following trick, inside the context menu component:

    Binding {
        target: __foreground
        property: "activeFocusOnPress"
        value: false
    }

This works for context menu implementations based on Popover (or ActionSelectionPopover), but unfortunately it doesn’t for the browser’s mobile implementation, which inherits Dialog, because Dialog also does this:

    Component.onCompleted: foreground.forceActiveFocus()

Not sure how to inhibit this behaviour.

> Not sure how to inhibit this behaviour.

Until we come up with a better solution, we can at the very least restore active focus on the webview when closing the context menu.

The problem has gotten worse with the latest release of the UITK (1.3.1778+16.04.20151217.1-0ubuntu1). With the fix for bug #1523828, popovers are now requesting active focus when they are shown, so the workaround I mentioned in comment #7 is useless.

In gmail, when long pressing on the compose field of a new message, the webview looses focus, which hides the OSK, which resizes the webview, which as a consequence dismisses the context menu. So the context menu has become unusable in gmail.

This bug was fixed in the package webbrowser-app - 0.23+16.04.20160112.1-0ubuntu1

---------------
webbrowser-app (0.23+16.04.20160112.1-0ubuntu1) xenial; urgency=medium

  [ CI Train Bot ]
  * Resync trunk. added: po/et.po

  [ Michael Sheldon ]
  * Dynamically load the thumbnailer image provider if it's available,
    otherwise fallback to mimetype images (LP: #1531179)

  [ Olivier Tilloy ]
  * Address a number of minor functional and code issues in the new
    downloads-in-browser functionality. (LP: #1528651)
  * Restore active focus on the webview after closing the context menu.
    (LP: #1526884)

 -- Olivier Tilloy <email address hidden> Tue, 12 Jan 2016 17:44:22 +0000